Kaijie Chen

Results 59 comments of Kaijie Chen

> Whoever merges this, please make sure to edit the commit message to match PR title if necessary (because GitHub uses the message from commit by default if the PR...

Thanks for the work @Xushaohong. Is it possible to delete those blocks directly instead of relying on OpenKeyCleanupService? I'm worrying about too much coupling here.

> Thx for the question~ @kaijchen > The case for OpenKeyCleanupService is not related to the logic here. They should be two cases as I mentioned in the DOC. Sorry...

Thanks @Xushaohong for updating the patch. Filtering uncommitted blocks in `verifyAndGetKeyLocations` is a good idea. For clearity, it's better to return a `Pair` instead, or create a inner class.

> yes, are you interested in updating it? I have created #3861 to update it.

There are failures in CI, please make sure CI pass before merge @kerneltime.

Thanks @xichen01 for the work, @duongkame and @sadanand48 for the review.

Sorry I have to revert this commit because it's making `ITestOzoneContractMkdir` flaky. Please take a look and submit another PR for this fix. @xichen01

> @kaijchen Are there some failed operations I can refer to them? So I can find out which tests are flaky. Here is one of the failing case, but not...

Thanks @Xushaohong for the improvement and @JacksonYao287 for the review. I have backported this fix into 1.3 branch.