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

   ## Description
   
   Reduce warning-log amplification when consensus subscription commit requests 
contain many rejected contexts.
   
   - Keep the receiver-level partial-commit summary at WARN.
   - Log rejected normal ACK context details at DEBUG while retaining WARN for 
internal NACK failures.
   - Log missing outstanding commit mappings at DEBUG because late or duplicate 
ACKs are already reported by the queue or summarized by the receiver.
   
   In the reported 1,000-table / 100,000-device workload, this reduces a 
typical commit from about 114 WARN lines to one summary WARN while preserving 
detailed diagnostics at DEBUG.
   
   ## Validation
   
   - `mvn spotless:check -pl iotdb-core/datanode`
   - `mvn checkstyle:check -pl iotdb-core/datanode -DskipTests` (0 violations)
   - `ConsensusSubscriptionCommitStateTest` passed 9/9 on the 2.0.11 
implementation branch.
   - The same targeted test on current master was blocked during compilation by 
unrelated stale SNAPSHOT cross-module errors in existing Pipe, Schema, and RPC 
sources; the test phase was not reached.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] added comments explaining the why and intent where it is not obvious.
   
   <hr>
   
   ##### Key changed classes
   
   - `ConsensusSubscriptionBroker`
   - `ConsensusSubscriptionCommitManager`


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to