wangxianghu commented on a change in pull request #1900:
URL: https://github.com/apache/hudi/pull/1900#discussion_r478423255



##########
File path: 
hudi-integ-test/src/main/java/org/apache/hudi/integ/testsuite/generator/GenericRecordFullPayloadGenerator.java
##########
@@ -43,22 +44,39 @@
  */
 public class GenericRecordFullPayloadGenerator implements Serializable {
 
-  public static final int DEFAULT_PAYLOAD_SIZE = 1024 * 10; // 10 KB
+  /**
+   * 10 KB.
+   */
+  public static final int DEFAULT_PAYLOAD_SIZE = 1024 * 10;
   private static Logger log = 
LoggerFactory.getLogger(GenericRecordFullPayloadGenerator.class);
   protected final Random random = new Random();
-  // The source schema used to generate a payload
+  /**
+   * The source schema used to generate a payload.

Review comment:
       > Why we should change these comment styles for fields?
   
   my bad,  that`s the coding guidelines of Alibaba. 
   rolled back already :)




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

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


Reply via email to