[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15406823#comment-15406823
]
Subru Krishnan commented on YARN-5406:
--
+1 on the latest patch. Thanks [~ellenfkh] for addressing our
[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15406502#comment-15406502
]
Hadoop QA commented on YARN-5406:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15404941#comment-15404941
]
Subru Krishnan commented on YARN-5406:
--
Thanks [~ellenfkh] for the patch and [~jianhe] for the review.
[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15403480#comment-15403480
]
Jian He commented on YARN-5406:
---
Also, should the .impl package be under the .store package ?
> In-memory
[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15403473#comment-15403473
]
Jian He commented on YARN-5406:
---
lgtm, not related to this patch.
I think mentioning the default value in
[
https://issues.apache.org/jira/browse/YARN-5406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15403001#comment-15403001
]
Hadoop QA commented on YARN-5406:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||