Zouxxyy commented on code in PR #7190:
URL: https://github.com/apache/hudi/pull/7190#discussion_r1020913952


##########
hudi-common/src/test/java/org/apache/hudi/common/testutils/HoodieCommonTestHarness.java:
##########
@@ -91,6 +91,12 @@ protected void initMetaClient() throws IOException {
     basePath = metaClient.getBasePath();
   }
 
+  protected void cleanMetaClient() {

Review Comment:
   > you could actually register this with `@AfterEach` ? so that subclasses 
won't have to call this
   
   There is no `set up` method provided in HoodieCommonTestHarness, the 
subclass controls the objects that need to be initialized, so the subclass 
controls the cleanup as well.



-- 
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]

Reply via email to