sijie commented on a change in pull request #4062: Delayed message delivery 
implementation
URL: https://github.com/apache/pulsar/pull/4062#discussion_r277191857
 
 

 ##########
 File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/LoadManager.java
 ##########
 @@ -134,9 +134,7 @@ static LoadManager create(final PulsarService pulsar) {
             // Assume there is a constructor with one argument of 
PulsarService.
             final Object loadManagerInstance = loadManagerClass.newInstance();
             if (loadManagerInstance instanceof LoadManager) {
-                final LoadManager casted = (LoadManager) loadManagerInstance;
-                casted.initialize(pulsar);
-                return casted;
+                return (LoadManager) loadManagerInstance;
 
 Review comment:
   Why this is changed? And how is it related to delayed delivery?

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