usberkeley commented on code in PR #11702:
URL: https://github.com/apache/hudi/pull/11702#discussion_r1698204702


##########
hudi-flink-datasource/hudi-flink/src/test/java/org/apache/hudi/sink/utils/MockStreamingRuntimeContext.java:
##########
@@ -47,21 +48,21 @@ public class MockStreamingRuntimeContext extends 
StreamingRuntimeContext {
   private int attemptNumber;
 
   public MockStreamingRuntimeContext(
-      boolean isCheckpointingEnabled,
-      int numParallelSubtasks,
-      int subtaskIndex) {
+          boolean isCheckpointingEnabled,

Review Comment:
   I found that the checkstyle.xml configuration does not recognize the 
indentation error when the function parameters wrap to a new line. Fixing this 
may cause a large number of files to need to be fixed.
   
   How about putting this indentation problem in the next PR?



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