[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-07 Thread Prabhu Joseph (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16787432#comment-16787432 ] Prabhu Joseph commented on YARN-9341: - Thanks [~eyang] and [~wilfreds]! > Reentrant lock() before try

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-07 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16787232#comment-16787232 ] Eric Yang commented on YARN-9341: - [~wilfreds] You are right. They have been handled correctly. Thank

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-07 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16787239#comment-16787239 ] Hudson commented on YARN-9341: -- FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #16157 (See

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-05 Thread Wilfred Spiegelenburg (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16785162#comment-16785162 ] Wilfred Spiegelenburg commented on YARN-9341: - So we do have one and also one

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-05 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16784984#comment-16784984 ] Eric Yang commented on YARN-9341: -

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-04 Thread Wilfred Spiegelenburg (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16783987#comment-16783987 ] Wilfred Spiegelenburg commented on YARN-9341: - An {{IllegalMonitorStateException}} can only

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-04 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16783671#comment-16783671 ] Eric Yang commented on YARN-9341: - [~Prabhu Joseph] When lock is moved outside of try, unlock may throw

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-04 Thread Prabhu Joseph (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16783366#comment-16783366 ] Prabhu Joseph commented on YARN-9341: - [~ajisakaa] Can you review this jira. The patch failures are

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-04 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16783268#comment-16783268 ] Hadoop QA commented on YARN-9341: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9341) Reentrant lock() before try

2019-03-03 Thread Prabhu Joseph (JIRA)
[ https://issues.apache.org/jira/browse/YARN-9341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16783062#comment-16783062 ] Prabhu Joseph commented on YARN-9341: - Reviewed below files which are using lock() in