hemantk-12 commented on PR #3958: URL: https://github.com/apache/ozone/pull/3958#issuecomment-1322544836
> Looks like there are a bunch of failures in the newly added test: > > https://github.com/apache/ozone/actions/runs/3510796319/jobs/5880950816 > > ``` > Error: Failures: > Error: TestOzoneManagerDoubleBuffer.testOzoneManagerDoubleBuffer:201 expected: <1> but was: <2> > Error: TestOzoneManagerDoubleBuffer.testOzoneManagerDoubleBuffer:208 expected: <3> but was: <4> > Error: TestOzoneManagerDoubleBuffer.testOzoneManagerDoubleBuffer:208 expected: <6> but was: <7> > Error: TestOzoneManagerDoubleBuffer.testOzoneManagerDoubleBuffer:208 expected: <10> but was: <11> > Error: TestOzoneManagerDoubleBuffer.testOzoneManagerDoubleBuffer:208 expected: <14> but was: <15> > [INFO] > Error: Tests run: 628, Failures: 5, Errors: 0, Skipped: 1 > ``` Yes, It worked fine on mac but seems like failing due to some race condition in workflow. I thought it will be fixed if we [disabled](https://github.com/apache/ozone/pull/3958/files#diff-195575b08642fad83fad37b0cbe2ba5fa39f9db550028ac540523de01ecff3a9R190) the daemon thread and [enable](https://github.com/apache/ozone/pull/3958/files#diff-195575b08642fad83fad37b0cbe2ba5fa39f9db550028ac540523de01ecff3a9R197) it after adding responses to the currentBuffer. But still failing. I'm gone extract out the flushing logic and test it independently. -- 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]
