[GitHub] [inlong] healchow commented on a diff in pull request #7402: [INLONG-7401][Manager] Fix the problem of partitions could not be created based on numPartitions

[via GitHub]

healchow commented on code in PR #7402:
URL: https://github.com/apache/inlong/pull/7402#discussion_r1113189263


##########
inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/resource/queue/pulsar/PulsarOperator.java:
##########
@@ -49,7 +49,7 @@
 public class PulsarOperator {
 
     private static final Logger LOGGER = 
LoggerFactory.getLogger(InlongClusterServiceImpl.class);
-    private static final int MAX_PARTITION = 100;
+    private static final int MAX_PARTITION = 1000;

Review Comment:
   Please add some comment for the value, such as:
   ```suggestion
       /**
        *The maximum number of partitions, which is an empirical value,
        * generally does not exceed 1000 in large clusters.
        */
       private static final int MAX_PARTITION = 1000;
   ```



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

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