hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Izaak Rubin (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-841) Consolidate multiple overloaded methods in HRegionInterface, HRegionServer
Date Wed, 20 Aug 2008 18:12:44 GMT

    [ https://issues.apache.org/jira/browse/HBASE-841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12624077#action_12624077

Izaak Rubin commented on HBASE-841:

The redundant HRegionServer methods pass on their redundancy to HRegion.  Do we also want
to trim the redundancy in HRegion?  A wider variety of classes use HRegion and would have
to be modified if we change that too - not that this is a problem, but do we want to leave
in some of the client convenience?

> Consolidate multiple overloaded methods in HRegionInterface, HRegionServer
> --------------------------------------------------------------------------
>                 Key: HBASE-841
>                 URL: https://issues.apache.org/jira/browse/HBASE-841
>             Project: Hadoop HBase
>          Issue Type: Improvement
>          Components: ipc, regionserver
>    Affects Versions: 0.2.0
>            Reporter: Jim Kellerman
>            Assignee: Izaak Rubin
>             Fix For: 0.3.0
> There are too many overloaded methods in HRegionServerInterface and consequently HRegionServer.
> These should be consolidated into one method per operation and the client should pass
the appropriate parameters to the server.
> On the server side, the single method should be able to handle parameters that are not
supplied, e.g.,
> - long values that are not supplied should be -1
> - boolean values should be supplied appropriately
> - objects that are not supplied should be passed as null
> All these overloaded methods eventually call the same method on the server side eventually.
Removing the overloads would make following control flow easier.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message