[GitHub] [hudi] danny0405 commented on a change in pull request #3972: [HUDI-2738] Remove the bucketAssignFunction useless context

[GitBox <gi...@apache.org>]

danny0405 commented on a change in pull request #3972:
URL: https://github.com/apache/hudi/pull/3972#discussion_r747291962



##########
File path: hudi-flink/src/test/java/org/apache/hudi/sink/utils/StreamWriteFunctionWrapper.java
##########
@@ -125,11 +124,10 @@ public StreamWriteFunctionWrapper(String tablePath, Configuration conf) throws E
     this.coordinatorContext = new MockOperatorCoordinatorContext(new OperatorID(), 1);
     this.coordinator = new StreamWriteOperatorCoordinator(conf, this.coordinatorContext);
     this.compactFunctionWrapper = new CompactFunctionWrapper(this.conf);
-    this.bucketAssignOperatorContext = new MockBucketAssignOperatorContext();
+    this.bucketAssignOperatorContext = new MockBucketAssignFunctionState();
     this.stateInitializationContext = new MockStateInitializationContext();

Review comment:
       Rename to `bucketAssignFunctionContext`.




-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org