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 2019/03/08 17:18:35 UTC

[hbase] branch master updated: HBASE-21416 - fix TestRegionInfoDisplay flaky test

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

stack pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new 8d3e751  HBASE-21416 - fix TestRegionInfoDisplay flaky test
8d3e751 is described below

commit 8d3e7517e6b6fb19ebc88d5e4e078fdfab43d84d
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));
   }