You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by "nsivabalan (via GitHub)" <gi...@apache.org> on 2026/06/25 03:04:57 UTC

Re: [I] Fix taskmanager concurrent registry [hudi]

nsivabalan commented on issue #15594:
URL: https://github.com/apache/hudi/issues/15594#issuecomment-4795450901

   This appears to be fixed in current master. The concurrent metric-timer registration race in `HoodieLockMetrics.createTimerForMetrics` was resolved by:
   
   - [HUDI-5041] PR #6968 (commit `c038e76d3dfb`) — wrapped the create-if-absent in `synchronized (REGISTRY_LOCK)`.
   - [MINOR] PR #8778 (commit `f04f95978406`) — added the exact double-checked-locking pattern that PR #7306 was proposing.
   
   Current code: [`HoodieLockMetrics.java#L101-L113`](https://github.com/apache/hudi/blob/master/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/metrics/HoodieLockMetrics.java#L101-L113). PR #7306 itself was closed unmerged since the equivalent fix had already landed via #8778. Closing this issue (and HUDI-5280) seems appropriate unless someone is still hitting the duplicate-registration error on a recent release.


-- 
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@hudi.apache.org

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