[ 
https://issues.apache.org/jira/browse/YARN-7200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17232833#comment-17232833
 ] 

Agshin Kazimli commented on YARN-7200:
--------------------------------------

[~snemeth] [~shuzirra] Thanks for the reviews.

As there are key differences how is the SLSRunner terminates, it's really hard 
to keep current config and be able to close the json file via old way. In this 
case, either I should change the exit logic or just ignore the the fact that 
BufferedWriter is static. SchedulerMetrics is extended by FifoSchedulerMetrics, 
FairSchedulerMetrics, CapacitySchedulerMetrics classes, which in case, I think, 
there is no problem, because, the SLS can only use one scheduler at a time.

> SLS generates a realtimetrack.json file but that file is missing the closing 
> ']'
> --------------------------------------------------------------------------------
>
>                 Key: YARN-7200
>                 URL: https://issues.apache.org/jira/browse/YARN-7200
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: scheduler-load-simulator
>            Reporter: Grant Sohn
>            Assignee: Agshin Kazimli
>            Priority: Minor
>              Labels: newbie, newbie++
>         Attachments: YARN-7200-branch-trunk.patch, realtimetrack.json, 
> snemeth-testing-20201113.zip
>
>
> File 
> hadoop-tools/hadoop-sls/src/main/java/org/apache/hadoop/yarn/sls/scheduler/SchedulerMetrics.java
>  shows:
> {noformat}
>   void tearDown() throws Exception {
>     if (metricsLogBW != null)  {
>       metricsLogBW.write("]");
>       metricsLogBW.close();
>     }
>     ....
> {noformat}
> So the exit logic is flawed.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to