[
https://issues.apache.org/jira/browse/HBASE-11579?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14072806#comment-14072806
]
Liu Shaohui commented on HBASE-11579:
-------------------------------------
Patch LGTM
> CopyTable should check endtime value only if != 0
> -------------------------------------------------
>
> Key: HBASE-11579
> URL: https://issues.apache.org/jira/browse/HBASE-11579
> Project: HBase
> Issue Type: Bug
> Components: mapreduce
> Affects Versions: 0.99.0, 0.98.4
> Reporter: Jean-Marc Spaggiari
> Assignee: Jean-Marc Spaggiari
> Attachments: HBASE-11579-v0-trunk.patch
>
>
> CopyTable automatically assign an endTime if startTime is specified and
> endTime is not:
> {code}
> if (startTime != 0) {
> scan.setTimeRange(startTime,
> endTime == 0 ? HConstants.LATEST_TIMESTAMP : endTime);
> }
> {code}
> However, we test if endTime is == 0 and exit before we get a chance to set
> the range:
> {code}
> if (startTime > endTime) {
> printUsage("Invalid time range filter: starttime=" + startTime + " >
> endtime=" + endTime);
> return false;
> }
> {code}
> So we need to check endTime only if it's != 0.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
