[GitHub] [ozone] umamaheswararao commented on pull request #3476: HDDS-6823. Intermittent failure in TestOzoneECClient#testExcludeOnDNMixed

[GitBox <gi...@apache.org>]

umamaheswararao commented on PR #3476:
URL: https://github.com/apache/ozone/pull/3476#issuecomment-1145107476

   LGTM, let's see if this fixed other PR test failures.
   I can only see pre allocated blocks vs lazily allocated. I think this will do 2 blocks preallocated compared to previous change.
   If preallocated 10 nodes will be used, otherwise it may reuse few nodes from previous. Anyway for this tests, preallocating is ok to have better predicted assertions. It would be good to document about this fact if that's true. ANyway for unblocking others I am going ahead to commit this test changes. Thanks a lot @kaijchen for quick investigation.


-- 
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: issues-unsubscribe@ozone.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org