You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@fluo.apache.org by GitBox <gi...@apache.org> on 2020/03/07 16:30:00 UTC
[GitHub] [fluo] jkosh44 opened a new pull request #1090: Remove race
condition from FluoAdminImpl.numWorkers
jkosh44 opened a new pull request #1090: Remove race condition from FluoAdminImpl.numWorkers
URL: https://github.com/apache/fluo/pull/1090
The pattern of checking if a ZNode exists and then getting it's
children contains a race condition. If the ZNode is deleted after
the existence check but before getting the children then a
NoNodeException will be thrown instead of returning 0.
Instead we skip the existence check and just return 0 when a
NoNodeException is thrown. This will avoid the race condition.
Fixes #1089
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [fluo] jkosh44 merged pull request #1090: Remove race condition
from FluoAdminImpl.numWorkers
Posted by GitBox <gi...@apache.org>.
jkosh44 merged pull request #1090: Remove race condition from FluoAdminImpl.numWorkers
URL: https://github.com/apache/fluo/pull/1090
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services