[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-10-03 Thread Robert Kanter (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16189957#comment-16189957 ] Robert Kanter commented on YARN-7207: - [~aw], did you have any more comments? Otherwis

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-28 Thread Robert Kanter (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16184437#comment-16184437 ] Robert Kanter commented on YARN-7207: - I agree that it's a bad idea and ideally the use

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Allen Wittenauer (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183572#comment-16183572 ] Allen Wittenauer commented on YARN-7207: Actually, let me expand on that a bit, bec

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Allen Wittenauer (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183566#comment-16183566 ] Allen Wittenauer commented on YARN-7207: bq. Single call of getLocalHost is pretty

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183554#comment-16183554 ] Yufei Gu commented on YARN-7207: There are different symptoms. Single call of getLocalHost

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Allen Wittenauer (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183527#comment-16183527 ] Allen Wittenauer commented on YARN-7207: If resolving the local hostname is slow, t

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183314#comment-16183314 ] Yufei Gu commented on YARN-7207: File YARN-7263 for #2. > Cache the local host name when

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-27 Thread Robert Kanter (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16182871#comment-16182871 ] Robert Kanter commented on YARN-7207: - +1 LGTM > Cache the local host name when gettin

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-26 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16180579#comment-16180579 ] Hadoop QA commented on YARN-7207: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-26 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16180473#comment-16180473 ] Yufei Gu commented on YARN-7207: Thanks [~rkanter] for the review. Uploaded patch v2 for yo

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-25 Thread Robert Kanter (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16179957#comment-16179957 ] Robert Kanter commented on YARN-7207: - Looks good overall, a few comments: # In {{TestR

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-25 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16178721#comment-16178721 ] Yufei Gu commented on YARN-7207: What did I do in my first patch? # Cache the RM proxy serv

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-20 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16174213#comment-16174213 ] Hadoop QA commented on YARN-7207: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-17 Thread Yufei Gu (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16169636#comment-16169636 ] Yufei Gu commented on YARN-7207: Thanks [~miklos.szeg...@cloudera.com], nice suggestion! >

[jira] [Commented] (YARN-7207) Cache the local host name when getting application list in RM

2017-09-15 Thread Miklos Szegedi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16168732#comment-16168732 ] Miklos Szegedi commented on YARN-7207: -- Thank you [~yufeigu] for reporting this. Pleas