[ https://issues.apache.org/jira/browse/HBASE-15283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15151699#comment-15151699 ]
Hudson commented on HBASE-15283: -------------------------------- SUCCESS: Integrated in HBase-1.2-IT #440 (See [https://builds.apache.org/job/HBase-1.2-IT/440/]) HBASE-15283 Revert to IOException in TimeRange constructor to maintain (jmhsieh: rev a8314301c28aa0738596f1dc2dd87f4d8dcb2a16) * hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java * hbase-client/src/main/java/org/apache/hadoop/hbase/client/Query.java > 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 > Components: Client > Affects Versions: 1.2.0, 1.3.0 > Reporter: Jonathan Hsieh > Assignee: Jonathan Hsieh > Fix For: 1.2.0, 1.3.0 > > Attachments: hbase-15283.branch-1.patch, hbase-15283.v2.branch-1.patch > > > 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)