You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/20 09:04:48 UTC

[GitHub] [pulsar] thetumbled opened a new issue, #17738: Duplicate data stored in loadbalance/brokers/

thetumbled opened a new issue, #17738:
URL: https://github.com/apache/pulsar/issues/17738

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Motivation
   
   i find that method getBundleStats return member lastStats,causing  duplicate data stored in ZK when jackson serializing LocalBrokerData,and the size of data in lastStats is pretty large,and i find that the method LocalBrokerData.getBundleStats is not used anywhere,i wonder whether we can deduplicate that. 
   
   org.apache.pulsar.policies.data.loadbalancer.LocalBrokerData#getBundleStats
   <img width="488" alt="image" src="https://user-images.githubusercontent.com/52550727/191214108-a77c09d9-59d2-4e13-85a2-b60aa622f425.png">
   
   
   ### Solution
   
   rename member lastStats to bundleStats.
   
   ### Alternatives
   
   ,
   
   ### Anything else?
   
   ,
   
   ### Are you willing to submit a PR?
   
   - [X] I'm willing to submit a PR!


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

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

[GitHub] [pulsar] mattisonchao commented on issue #17738: Duplicate data stored in loadbalance/brokers/

Posted by GitBox <gi...@apache.org>.
mattisonchao commented on issue #17738:
URL: https://github.com/apache/pulsar/issues/17738#issuecomment-1257365161

   Thanks a lot for your contribution.


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

[GitHub] [pulsar] nodece closed issue #17738: Duplicate data stored in loadbalance/brokers/

Posted by GitBox <gi...@apache.org>.
nodece closed issue #17738: Duplicate data stored in loadbalance/brokers/
URL: https://github.com/apache/pulsar/issues/17738


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

[GitHub] [pulsar] thetumbled commented on issue #17738: Duplicate data stored in loadbalance/brokers/

Posted by GitBox <gi...@apache.org>.
thetumbled commented on issue #17738:
URL: https://github.com/apache/pulsar/issues/17738#issuecomment-1286523199

   > Thanks a lot for your contribution.
   
   PTAL


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