You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "BukrosSzabolcs (via GitHub)" <gi...@apache.org> on 2025/08/28 08:57:07 UTC

[PR] Fix AccumuloVFSClassLoader race condition [accumulo]

BukrosSzabolcs opened a new pull request, #5835:
URL: https://github.com/apache/accumulo/pull/5835

   Run into a race condition:
   
   - ThreadA sets localLoader to null
   - ThreadA waits for lock
   - ThreadB has the lock
   - ThreadB initializes loader and returns
   - ThreadA gets the lock
   - ThreadA finds that loader is not null, but localLoader is so turns into an infinite loop and prevents master/ts init
   
   We should set localLoader if we find someone else already initialized loader to prevent this.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


Re: [PR] Fix AccumuloVFSClassLoader race condition [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner merged PR #5835:
URL: https://github.com/apache/accumulo/pull/5835


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


Re: [PR] Fix AccumuloVFSClassLoader race condition [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #5835:
URL: https://github.com/apache/accumulo/pull/5835#issuecomment-3237579989

   Merged this to main -sours because the class does not exist in main, it was moved to https://github.com/apache/accumulo-classloaders.  I took a quick look there and did not see a similar problem, just looked the beginning of while loops looking for a similar pattern.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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