[
https://issues.apache.org/jira/browse/HBASE-5694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13584412#comment-13584412
]
Guido Serra aka Zeph commented on HBASE-5694:
---------------------------------------------
[~uws] if this is actually the patch (taken from above)
{code}
--- ThriftServer.java.orig 2012-04-01 23:41:16.881172406 +0200
+++ ThriftServer.java 2012-04-01 23:41:30.177238337 +0200
@@ -477,8 +477,8 @@
get.addColumn(famAndQf[0], famAndQf[1]);
}
}
- get.setTimeRange(Long.MIN_VALUE, timestamp);
}
+ get.setTimeRange(Long.MIN_VALUE, timestamp);
gets.add(get);
}
Result[] result = table.get(gets);
{code}
it is the wrong behavior that I'm getting, as it is inconsistent with the
scannerOpenWithScan
we shall not use the setTimeRange but the setTimestamp... as the signature in
Thrift states:
{code}
471 * Get the specified columns for the specified table and rows at the
specified
472 * timestamp. Returns an empty list if no rows exist.
{code}
and not a range scan from Long.MIN_VALUE to timestamp as implemented above
> getRowsWithColumnsTs() in Thrift service handles timestamps incorrectly
> -----------------------------------------------------------------------
>
> Key: HBASE-5694
> URL: https://issues.apache.org/jira/browse/HBASE-5694
> Project: HBase
> Issue Type: Bug
> Components: Thrift
> Affects Versions: 0.92.1
> Reporter: Wouter Bolsterlee
> Fix For: 0.94.0
>
> Attachments: HBASE-5694.patch, HBASE-5694-trunk-20120402.patch
>
>
> The getRowsWithColumnsTs() method in the Thrift interface only applies the
> timestamp if columns are explicitly specified. However, this method also
> allows for columns to be unspecified (this is even used internally to
> implement e.g. getRows()). The cause of the bug is a minor scoping issue: the
> time range is set inside a wrong if statement.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
