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