[jira] [Commented] (YARN-10185) Container executor fields should be volatile

2020-04-03 Thread Adam Antal (Jira)
[ https://issues.apache.org/jira/browse/YARN-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17074416#comment-17074416 ] Adam Antal commented on YARN-10185: --- I think this can be closed based on the comments above. Thanks

[jira] [Commented] (YARN-10185) Container executor fields should be volatile

2020-03-09 Thread Eric Badger (Jira)
[ https://issues.apache.org/jira/browse/YARN-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17055446#comment-17055446 ] Eric Badger commented on YARN-10185: I agree with [~pbacsko]. Since initialization is synchronized

[jira] [Commented] (YARN-10185) Container executor fields should be volatile

2020-03-09 Thread Denes Gerencser (Jira)
[ https://issues.apache.org/jira/browse/YARN-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17055383#comment-17055383 ] Denes Gerencser commented on YARN-10185: Thanks [~pbacsko] for looking into this! The whole call

[jira] [Commented] (YARN-10185) Container executor fields should be volatile

2020-03-09 Thread Peter Bacsko (Jira)
[ https://issues.apache.org/jira/browse/YARN-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17055089#comment-17055089 ] Peter Bacsko commented on YARN-10185: - I don't think that {{volatile}} is needed at all. The

[jira] [Commented] (YARN-10185) Container executor fields should be volatile

2020-03-09 Thread Adam Antal (Jira)
[ https://issues.apache.org/jira/browse/YARN-10185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17054783#comment-17054783 ] Adam Antal commented on YARN-10185: --- [~ebadger] can you please take a look at this? I wonder how this