You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2020/07/14 10:16:42 UTC

[hbase] branch branch-2.1 updated: HBASE-24693 regioninfo#isLast() should point to EMPTY_END_ROW (readability improvement)

This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new e618b1c  HBASE-24693 regioninfo#isLast() should point to EMPTY_END_ROW (readability improvement)
e618b1c is described below

commit e618b1cdd8b0dfd23c7cf573670b9445175b9feb
Author: Bo Cui <cu...@163.com>
AuthorDate: Tue Jul 14 15:38:23 2020 +0530

    HBASE-24693 regioninfo#isLast() should point to EMPTY_END_ROW (readability improvement)
    
    Closes #2051
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
    Signed-off-by: Viraj Jasani <vj...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java        | 2 +-
 hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java
index 2f9e88d..cc138ce 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionInfo.java
@@ -769,7 +769,7 @@ public interface RegionInfo {
    * @return True if this is last Region in Table
    */
   default boolean isLast() {
-    return Bytes.equals(getEndKey(), HConstants.EMPTY_START_ROW);
+    return Bytes.equals(getEndKey(), HConstants.EMPTY_END_ROW);
   }
 
   /**
diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java
index b41c230..1a713fc 100644
--- a/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java
+++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java
@@ -614,7 +614,7 @@ public final class HConstants {
   /**
    * Last row in a table.
    */
-  public static final byte [] EMPTY_END_ROW = EMPTY_START_ROW;
+  public static final byte [] EMPTY_END_ROW = EMPTY_BYTE_ARRAY;
 
   /**
     * Used by scanners and others when they're trying to detect the end of a