cshannon commented on PR #4415: URL: https://github.com/apache/accumulo/pull/4415#issuecomment-2040157051
The latest update [demonstrates](https://github.com/apache/accumulo/blob/e0fe9e49b9bc00f101afa6a7fa302f792d53b55b/test/src/main/java/org/apache/accumulo/test/ample/TestAmpleIT.java#L203) using the new test API that the fate unspittable marker is written for an unsplittlable tablet. It essentially replicates parts of the test in [LargeSplitRowIT](https://github.com/apache/accumulo/blob/eea8ce48e2d5d7bd5c6b69e05c6824f697d8de0a/test/src/main/java/org/apache/accumulo/test/LargeSplitRowIT.java#L262) that runs with the real metadata table. One really nice side effect of this that I didn't think about is that you can use break points. Normally when testing with ITs the FATE code is running in the Manager JVM so you would need to open up a remote debug port to use break points. With this change the test repo code is running in the unit test JVM so you can easily step through with break points the code which makes debugging a lot easier. I think we need both types of tests going forward but certainly having this new API will be nice to test things that are not easy with a normal IT. -- 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. To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
