[GitHub] [pinot] xiangfu0 commented on a change in pull request #7308: Enable maven checkstyle plugins by default for all new pinot components and fix pinot-spi styling

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

xiangfu0 commented on a change in pull request #7308:
URL: https://github.com/apache/pinot/pull/7308#discussion_r689962746



##########
File path: pinot-spi/src/main/java/org/apache/pinot/spi/config/table/assignment/InstanceReplicaGroupPartitionConfig.java
##########
@@ -41,7 +41,8 @@
   @JsonPropertyDescription("Number of partitions for replica-group based selection, do not partition the replica-group (1 partition) if not specified")
   private final int _numPartitions;
 
-  @JsonPropertyDescription("Number of instances per partition (within a replica-group) for replica-group based selection, select all instances if not specified")
+  @JsonPropertyDescription(
+      "Number of instances per partition (within a replica-group) for replica-group based selection, select all instances if not specified")

Review comment:
       Although the error msg shows don't go beyond 120, the actual limit is 160 :p




-- 
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@pinot.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org