[GitHub] [hbase] arshadmohammad commented on a change in pull request #2785: HBASE-25371: When openRegion fails during initial verification(before…

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org