You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by op...@apache.org on 2019/03/11 11:46:02 UTC

[hbase] 16/24: HBASE-21416 - fix TestRegionInfoDisplay flaky test

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

openinx pushed a commit to branch HBASE-21879
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit 31902a098c5a27c59d67fd41c39042db46e15d49
Author: Norbert Kalmar <nk...@cloudera.com>
AuthorDate: Thu Mar 7 11:56:10 2019 +0100

    HBASE-21416 - fix TestRegionInfoDisplay flaky test
---
 .../java/org/apache/hadoop/hbase/client/TestRegionInfoDisplay.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestRegionInfoDisplay.java b/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestRegionInfoDisplay.java
index 1a6f2f7..0800acd 100644
--- a/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestRegionInfoDisplay.java
+++ b/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestRegionInfoDisplay.java
@@ -68,12 +68,13 @@ public class TestRegionInfoDisplay {
     RegionState state = RegionState.createForTesting(convert(ri), RegionState.State.OPEN);
     String descriptiveNameForDisplay =
         RegionInfoDisplay.getDescriptiveNameFromRegionStateForDisplay(state, conf);
-    checkDescriptiveNameEquality(descriptiveNameForDisplay,state.toDescriptiveString(), startKey);
+    String originalDescriptive = state.toDescriptiveString();
+    checkDescriptiveNameEquality(descriptiveNameForDisplay, originalDescriptive, startKey);
 
     conf.setBoolean("hbase.display.keys", true);
     Assert.assertArrayEquals(endKey, RegionInfoDisplay.getEndKeyForDisplay(ri, conf));
     Assert.assertArrayEquals(startKey, RegionInfoDisplay.getStartKeyForDisplay(ri, conf));
-    Assert.assertEquals(state.toDescriptiveString(),
+    Assert.assertEquals(originalDescriptive,
         RegionInfoDisplay.getDescriptiveNameFromRegionStateForDisplay(state, conf));
   }