devmadhuu commented on PR #10549:
URL: https://github.com/apache/ozone/pull/10549#issuecomment-4891724024

   > Sure, here is the benchmark figures based on test:
   > Approach   Perf Test Execution Time        Notes
   > Before (master baseline)   82.1 s  Delete-all + re-insert 
(replaceUnhealthyContainerRecordsAtomically)
   > After this PR      18.4 s  Insert / update / delete-stale only 
(syncUnhealthyContainerRecordsAtomically)
   > 
   > Improvement vs original: ~4.5× faster (82s → 18s)
   
   Are the above performance figures from previous insert and delete test where 
you added "`@Slow("HDDS-15582")`" ? Also I don't see you removed this tag if 
performance improved ?


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