[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15369715#comment-15369715
]
Hudson commented on YARN-4200:
--
SUCCESS: Integrated in Hadoop-trunk-Commit #10074 (See
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088391#comment-15088391
]
Vrushali C commented on YARN-4200:
--
+ 1, thanks [~gtCarrera9]
> Refactor reader classes in storage to
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088396#comment-15088396
]
Li Lu commented on YARN-4200:
-
If there are no further comments I'll commit this patch tomorrow morning.
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088358#comment-15088358
]
Li Lu commented on YARN-4200:
-
Any other concerns on this patch? All check style issues appear to be
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083932#comment-15083932
]
Vrushali C commented on YARN-4200:
--
Hi [~gtCarrera]
The patch looks good but I believe it's outdated now
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083933#comment-15083933
]
Li Lu commented on YARN-4200:
-
Sure. Will do that soon. Thanks!
> Refactor reader classes in storage to nest
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083968#comment-15083968
]
Hadoop QA commented on YARN-4200:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084104#comment-15084104
]
Hadoop QA commented on YARN-4200:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084040#comment-15084040
]
Hadoop QA commented on YARN-4200:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084090#comment-15084090
]
Vrushali C commented on YARN-4200:
--
Thanks [~gtCarrera9]! Patch looks good. There is an existing reader
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084102#comment-15084102
]
Vrushali C commented on YARN-4200:
--
I see, thanks!
> Refactor reader classes in storage to nest under
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084097#comment-15084097
]
Li Lu commented on YARN-4200:
-
Oh I believe right now we're not marking all ATS v2 related APIs to be public,
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14954227#comment-14954227
]
Hadoop QA commented on YARN-4200:
-
\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/YARN-4200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14903285#comment-14903285
]
Li Lu commented on YARN-4200:
-
Will flip the code quickly when there is no other interference. Right now our
14 matches
Mail list logo