peterxcli commented on code in PR #9234:
URL: https://github.com/apache/ozone/pull/9234#discussion_r2502543344
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/OzoneRpcClientTests.java:
##########
@@ -4650,13 +4656,21 @@ private void
checkExceptedResultForVersioningTest(String volumeName,
cluster.getOzoneManager().awaitDoubleBufferFlush();
if (expectedCount == 1) {
+ // Wait for deleted key to become visible in deletedTable
Review Comment:
the only reason I can came up is the
https://github.com/apache/ozone/commit/22fa6f5ea2e0ed8034096147534b4089f320ef87
set short period of time for the block deletion service and dir deletion service
```diff
static void startCluster(OzoneConfiguration conf, MiniOzoneCluster.Builder
builder) throws Exception {
...
+ conf.setTimeDuration(OZONE_BLOCK_DELETING_SERVICE_INTERVAL, 1,
TimeUnit.SECONDS);
+ conf.setTimeDuration(OZONE_DIR_DELETING_SERVICE_INTERVAL, 1,
TimeUnit.SECONDS);
...
}
```
which might introduce the between the deletion service and the normal
eg.
1. double buffer write record to deleted table
2. deletion service triggered, clean up the record in the deleted table
3. so the test assertion cant see records inside the deleted table
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]