You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2022/10/29 16:53:28 UTC

[GitHub] [iotdb] CRZbulabula opened a new pull request, #7799: [IOTDB-4796] Remove LoadStatistics consensus feature

CRZbulabula opened a new pull request, #7799:
URL: https://github.com/apache/iotdb/pull/7799

   The LoadStatistics should not be consensus among the ConfigNode-Group for the following reasons:
   
   1. They are easy to recompute within a short time.
   2. The updation of LoadStatistics in the future might cause compatibility problems in the future.
   
   Therefore, the LoadStatistics consensus feature will be abolished.
   
   Changes in this PR:
   
   1. The LoadStatistics consensus feature is abolished.
   2. Adapt LoadStatistics check in new framework i.e. cache previousStatistics and calculate currentStatistics at the same time, thus the LoadManager can detect whether some LoadStatistics is changed.
   3. Add formatted info level log in LoadManager when some LoadStatistics is changed. It's valuable to record the cluster load information and this process doesn't trigger frequently.


-- 
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: reviews-unsubscribe@iotdb.apache.org

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


[GitHub] [iotdb] Beyyes merged pull request #7799: [IOTDB-4796] Remove LoadStatistics consensus feature

Posted by GitBox <gi...@apache.org>.
Beyyes merged PR #7799:
URL: https://github.com/apache/iotdb/pull/7799


-- 
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: reviews-unsubscribe@iotdb.apache.org

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