xichen01 commented on code in PR #6335:
URL: https://github.com/apache/ozone/pull/6335#discussion_r1514284242
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/client/rpc/TestOzoneRpcClientWithRatis.java:
##########
@@ -323,7 +323,8 @@ public void testParallelDeleteBucketAndCreateKey() throws
IOException,
omSM.getHandler().setInjector(injector);
thread1.start();
thread2.start();
- Thread.sleep(2000);
+ // Wait long enough for createKey's preExecute to finish executing
+ Thread.sleep(10000);
Review Comment:
Yes, but this needs to modify more codes. We need to know the `preExecture`
has been executed completely, but the `preExecture` is a `prviate` method (or
we need to add some flag to watch), I don't want ot modify it just for only one
testing.
Add sleeping time is a dirty but quick solution. How do you think?
--
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]
