You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2017/09/01 19:36:41 UTC
hbase git commit: HBASE-18739 Make all TimeRange Constructors
InterfaceAudience Private
Repository: hbase
Updated Branches:
refs/heads/master 19bb4ef48 -> e1eb53296
HBASE-18739 Make all TimeRange Constructors InterfaceAudience Private
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e1eb5329
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e1eb5329
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e1eb5329
Branch: refs/heads/master
Commit: e1eb53296d984ea34cb5c2db5289412062b0c753
Parents: 19bb4ef
Author: Michael Stack <st...@apache.org>
Authored: Fri Sep 1 11:28:58 2017 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Fri Sep 1 12:36:31 2017 -0700
----------------------------------------------------------------------
.../org/apache/hadoop/hbase/io/TimeRange.java | 23 +++++++++++++++-----
1 file changed, 17 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/e1eb5329/hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java
----------------------------------------------------------------------
diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java
index 764b2a0..0d418d4 100644
--- a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java
+++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/TimeRange.java
@@ -29,7 +29,8 @@ import org.apache.hadoop.hbase.util.Bytes;
* Evaluated according to minStamp <= timestamp < maxStamp
* or [minStamp,maxStamp) in interval notation.
* <p>
- * Only used internally; should not be accessed directly by clients.
+ * Can be returned and read by clients. Should not be directly created by clients.
+ * Thus, all constructors are purposely @InterfaceAudience.Private.
*<p>Immutable. Thread-safe.
*/
@InterfaceAudience.Public
@@ -43,9 +44,11 @@ public class TimeRange {
/**
* Default constructor.
* Represents interval [0, Long.MAX_VALUE) (allTime)
- * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above
+ * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above and may be
+ * changed to private or removed in 3.0.
*/
@Deprecated
+ @InterfaceAudience.Private
public TimeRange() {
this(INITIAL_MIN_TIMESTAMP, INITIAL_MAX_TIMESTAMP);
}
@@ -53,9 +56,11 @@ public class TimeRange {
/**
* Represents interval [minStamp, Long.MAX_VALUE)
* @param minStamp the minimum timestamp value, inclusive
- * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above
+ * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above and may be
+ * changed to private or removed in 3.0.
*/
@Deprecated
+ @InterfaceAudience.Private
public TimeRange(long minStamp) {
this(minStamp, INITIAL_MAX_TIMESTAMP);
}
@@ -63,9 +68,11 @@ public class TimeRange {
/**
* Represents interval [minStamp, Long.MAX_VALUE)
* @param minStamp the minimum timestamp value, inclusive
- * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above
+ * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above and may be
+ * changed to private or removed in 3.0.
*/
@Deprecated
+ @InterfaceAudience.Private
public TimeRange(byte [] minStamp) {
this(Bytes.toLong(minStamp));
}
@@ -74,9 +81,11 @@ public class TimeRange {
* Represents interval [minStamp, maxStamp)
* @param minStamp the minimum timestamp, inclusive
* @param maxStamp the maximum timestamp, exclusive
- * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above
+ * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above and may be
+ * changed to private or removed in 3.0.
*/
@Deprecated
+ @InterfaceAudience.Private
public TimeRange(byte [] minStamp, byte [] maxStamp) {
this(Bytes.toLong(minStamp), Bytes.toLong(maxStamp));
}
@@ -86,9 +95,11 @@ public class TimeRange {
* @param minStamp the minimum timestamp, inclusive
* @param maxStamp the maximum timestamp, exclusive
* @throws IllegalArgumentException if either <0,
- * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above
+ * @deprecated This is made @InterfaceAudience.Private in the 2.0 line and above and may be
+ * changed to private or removed in 3.0.
*/
@Deprecated
+ @InterfaceAudience.Private
public TimeRange(long minStamp, long maxStamp) {
check(minStamp, maxStamp);
this.minStamp = minStamp;