priyeshkaratha commented on PR #10100:
URL: https://github.com/apache/ozone/pull/10100#issuecomment-4307419271

   @sumitagrawl @ChenSammi @szetszwo 
   
   The `ozone.scm.ha.ratis.snapshot.threshold` default of 1000 is smaller than 
the `ozone.scm.ha.ratis.server.snapshot.creation.gap` default of 1024. Because 
snapshot.creation.gap is the gate that Ratis applies to any manual snapshot API 
call (exactly the kind SCMHATransactionBufferMonitorTask makes), and because 
the auto-trigger always fires 24 entries earlier, the buffer monitor task never 
successfully triggers a snapshot under default configuration. It incurs an 
unnecessary Ratis RPC round-trip every 60 seconds that always goes nowhere, and 
any issue with the auto-trigger path has no time-based fallback as originally 
intended.
   
   The fix is to either:
   
   - Raise `ozone.scm.ha.ratis.snapshot.threshold` to be strictly greater than 
`ozone.scm.ha.ratis.server.snapshot.creation.gap` (e.g., threshold = 2000, gap 
= 1024) so that the time-based monitor becomes the primary snapshot trigger for 
moderate transaction volumes, with auto-trigger as the backstop.
   
   - Lower `ozone.scm.ha.ratis.server.snapshot.creation.gap` below threshold 
(e.g., gap = 1, threshold = 1000) so the monitor can trigger aggressively in 
low-traffic situations without waiting for the auto-trigger.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to