You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Jonathan Hsieh (JIRA)" <ji...@apache.org> on 2016/02/17 20:14:18 UTC
[jira] [Created] (HBASE-15283) Revert to IOException in TimeRange
constructor to maintain API compat in 1.x line
Jonathan Hsieh created HBASE-15283:
--------------------------------------
Summary: Revert to IOException in TimeRange constructor to maintain API compat in 1.x line
Key: HBASE-15283
URL: https://issues.apache.org/jira/browse/HBASE-15283
Project: HBase
Issue Type: Bug
Reporter: Jonathan Hsieh
>From the 1.2 rc3 conversation:
[~busbey]
{quote}
* TimeRange's constructor with a min/max came in HBASE-14355. The
constructor changed from IOException in the case of min > max to
IllegalArgumentException. That's probably a more accurate exception,
but it's a substantial source compat and behavior change for
downstream. I've flagged the issue as incompatible and asked for a
release note. For some reason the javadocs for TimeRange claim it's
"for internal use only" so maybe it shouldn't have been IA.Public in
the first place.
{quote}
Jon:
{quote}
A user could trigger the TimeRange constructed exception change by with the Get#setTimeRange/Scan#setTimeRange/Increment#setTimeRange apis. The signature in these classes don't change. (master [1], branch-1.2[2], tag 1.1.3[3])
Doing a little digging, it is worth noting that no public APIs take time range as an argument (HRegion, TimeRangeTracker do but they are private), however a few public metrics return it (Get, Scan, Increment). The comment also basically says don't use the constructor.
{quote}
[1]https://github.com/apache/hbase/blob/master/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Get.java#L205
[2]https://github.com/apache/hbase/blob/branch-1.2/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Get.java#L195
[3] https://github.com/apache/hbase/blob/rel/1.1.3/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Get.java#L191
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)