[
https://issues.apache.org/jira/browse/YARN-3879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16606294#comment-16606294
]
Vrushali C commented on YARN-3879:
----------------------------------
Patch looks good overall. If you are going to update it, then I would suggest
using File.separator instead of an actual "/" . If the patch does not need any
updating, then let's leave it.
Overall +1 on the patch
> [Storage implementation] Create HDFS backing storage implementation for ATS
> reads
> ---------------------------------------------------------------------------------
>
> Key: YARN-3879
> URL: https://issues.apache.org/jira/browse/YARN-3879
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: timelineserver
> Reporter: Tsuyoshi Ozawa
> Assignee: Abhishek Modi
> Priority: Major
> Labels: YARN-5355
> Attachments: YARN-3879-YARN-7055.001.patch, YARN-3879.001.patch,
> YARN-3879.002.patch, YARN-3879.003.patch, YARN-3879.004.patch
>
>
> Reader version of YARN-3841
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
