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

   ## Description
   
   - Remove the custom Ratis RPC type, gRPC factory, and log appender 
introduced in #18118.
   - Restore the default Ratis gRPC log appender so 
region_migration_speed_limit_bytes_per_second no longer affects Ratis snapshot 
transfers.
   - Keep region migration rate limiting for IoTConsensus and IoTConsensusV2 
unchanged.
   
   ## Motivation
   
   The current Ratis hook does not make the migration speed limit effective in 
practice. Correctly enforcing the limit requires broader integration with the 
Ratis transfer path, so this PR removes the partial Ratis behavior until that 
work is implemented properly.
   
   ## Testing
   
   - mvn spotless:apply -pl iotdb-core/consensus
   - mvn test -pl iotdb-core/consensus: all 27 Ratis tests passed. The overall 
goal reported 13 unrelated IoTConsensus errors because the test machine had 
only 7.9 GiB free and triggered DiskSpaceInsufficientException.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   
   <hr>
   
   ##### Key changed/added classes (or packages if there are too many classes) 
in this PR
   
   - org.apache.iotdb.consensus.ratis.RatisConsensus
   - Removed RateLimitedGrpcRpcType, RateLimitedGrpcFactory, and 
RateLimitedGrpcLogAppender.
   


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