[GitHub] [pulsar] RobertIndie commented on a change in pull request #13239: [pulsar-client]Add getNumPartitions method into PartitionedProducerImpl

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

RobertIndie commented on a change in pull request #13239:
URL: https://github.com/apache/pulsar/pull/13239#discussion_r767360344



##########
File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PartitionedProducerImpl.java
##########
@@ -438,4 +438,13 @@ public Timeout getPartitionsAutoUpdateTimeout() {
         return partitionsAutoUpdateTimeout;
     }
 
+    /**
+     * Return the number of partitions per topic.
+     *
+     * @return the number of partitions per topic.
+     */
+    public int getNumOfPartitions() {

Review comment:
       Do we need to expose this method to the `Producer` interface? I think we should not let the user call this method in the implementation class directly.




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

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