hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jim Kellerman (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-40) [hbase] Add a method of getting multiple (but not all) cells for a row at once
Date Mon, 10 Mar 2008 20:54:48 GMT

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

Jim Kellerman commented on HBASE-40:

Review of patch

- Two new APIs are for getRow are added to HRegionInterface, but only one is used. HTable
converts one form into the other.
  public SortedMap<Text, Cell> getRow(final Text row) throws IOException {
    return getRow(row, HConstants.LATEST_TIMESTAMP);

- one implementation of getFull in HRegion is commented out. Just remove it:
  // public Map<Text, Cell> getFull(Text row) throws IOException {
  //   return getFull(row, HConstants.LATEST_TIMESTAMP);
  // }

- If one of the APIs for getRow is removed from HRegionInterface, it should be removed from
HRegionServer as well.

- In HRS.getRow, the following could be simplified:
      Set<Text> columnSet = new HashSet<Text>();
      if (columns != null) {
        for (int i = 0; i < columns.length; i++) {


      Set<Text> columnSet = new HashSet<Text>();
      if (columns != null) {

not terribly important, but FYI.

If at least the first three changes are made, +1.

> [hbase] Add a method of getting multiple (but not all) cells for a row at once
> ------------------------------------------------------------------------------
>                 Key: HBASE-40
>                 URL: https://issues.apache.org/jira/browse/HBASE-40
>             Project: Hadoop HBase
>          Issue Type: New Feature
>          Components: client
>            Reporter: Bryan Duxbury
>            Assignee: Bryan Duxbury
>            Priority: Minor
>         Attachments: 40.patch
> We should have the ability to return some but not all cells from a row at once. There
are likely to be a number of situations when getFull will return much more data than needed,
but using individual get calls would likely be too small. This method should support returning
a specific list of columns all at once.
> {code}
> Map<Text, byte[]> results = table.getMulti(new Text[]{cellA, cellB, cellC}, timestamp);
> {code}

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

View raw message