sjlee commented on PR #7828:
URL: https://github.com/apache/hadoop/pull/7828#issuecomment-3119587988

   Thanks for the follow-up, @shameersss1.
   
   Looking at the exception stack trace from the JIRA, it is `host.attributes` 
that was concurrently modified, isn't it? It is `host.attributes` that needs to 
be protected. `newNodeToAttributesMap` is not a factor as it is a local 
variable and thus not shared.
   
   And, it is also the logging line itself (l.748) that is an unsafe read. IMO, 
the right fix is to protect the logging line (l.748) with the read lock. Please 
let me know.


-- 
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: common-issues-unsubscr...@hadoop.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to