You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/06/15 04:56:33 UTC
[GitHub] [druid] gianm opened a new pull request, #12658: Fix thread-unsafe emitter usage in SeekableStreamSupervisorStateTest.
gianm opened a new pull request, #12658:
URL: https://github.com/apache/druid/pull/12658
The TestEmitter is used from different threads without concurrency control. This patch makes the emitter thread-safe.
I think this fixes #12649, since:
1. Without the patch, the test generally fails once every 10–20 runs on my machine. With the patch, I've done 200 runs and it hasn't failed yet.
2. It's believable that the ConcurrentModificationException is due to the thread-unsafe emitter.
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] gianm merged pull request #12658: Fix thread-unsafe emitter usage in SeekableStreamSupervisorStateTest.
Posted by GitBox <gi...@apache.org>.
gianm merged PR #12658:
URL: https://github.com/apache/druid/pull/12658
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org