arshadmohammad commented on a change in pull request #2785:
URL: https://github.com/apache/hbase/pull/2785#discussion_r546698663



##########
File path: 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionOpen.java
##########
@@ -135,4 +139,36 @@ public void testNonExistentRegionReplica() throws 
Exception {
     }
     fail("Should have thrown IOE when attempting to open a non-existing 
region.");
   }
+
+  /*
+   * Before applying the fix run this test case, error message
+   * "java.io.IOException: The new max sequence id 1 is less than the old max 
sequence id 17"
+   * can be found the log.
+   * After applying the fix run this test case above message should not be 
logged.
+   */
+  //@Test(timeout = 60000)
+  @Ignore

Review comment:
       I was thinking not to run that test case in the CI as it is not doing 
any assert, will be of no use In CI.
   It was written to reproduce the issue. I think better to remove it from PR.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to