arina-ielchiieva commented on a change in pull request #1988: DRILL-7590: 
Refactor plugin registry
URL: https://github.com/apache/drill/pull/1988#discussion_r385084998
 
 

 ##########
 File path: common/src/main/java/org/apache/drill/common/PlanStringBuilder.java
 ##########
 @@ -56,20 +84,30 @@ public PlanStringBuilder field(String key, Object value) {
     return this;
   }
 
+  /**
+   * Displays a numeric field: {@code size=10}.
+   */
   public PlanStringBuilder field(String key, int value) {
     startField(key);
     buf.append(value);
     return this;
   }
 
+  /**
+   * Displays a character in Java-quoted format: {@code delimiter="\n"}.
+   * @param key
 
 Review comment:
   Please add descriptions, sorry for asking that, it's just IDE warnings are 
annoying when descriptions are not missing.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to