[jira] [Updated] (YARN-7132) FairScheduler.initScheduler() contains a surprising unary plus

2017-08-30 Thread Yeliang Cang (JIRA) 

 [ 
https://issues.apache.org/jira/browse/YARN-7132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yeliang Cang updated YARN-7132:
---
Attachment: YARN-7132.001.patch

Thanks [~templedf]! I have submitted a patch, and please check it!

> FairScheduler.initScheduler() contains a surprising unary plus
> --
>
> Key: YARN-7132
> URL: https://issues.apache.org/jira/browse/YARN-7132
> Project: Hadoop YARN
>  Issue Type: Improvement
>  Components: fairscheduler
>Affects Versions: 3.0.0-alpha4
>Reporter: Daniel Templeton
>Priority: Minor
>  Labels: newbie
> Attachments: YARN-7132.001.patch
>
>
> The method contains the following code:{code}
> LOG.warn(FairSchedulerConfiguration.UPDATE_INTERVAL_MS
> + " is invalid, so using default value "
> + +FairSchedulerConfiguration.DEFAULT_UPDATE_INTERVAL_MS
> + " ms instead");{code}
> Note the beginning of the third line.  One of those plusses should be deleted 
> so that no one else spends cycles trying to understand why it even compiles.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

[jira] [Updated] (YARN-7132) FairScheduler.initScheduler() contains a surprising unary plus

2017-08-30 Thread Daniel Templeton (JIRA) 

 [ 
https://issues.apache.org/jira/browse/YARN-7132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Templeton updated YARN-7132:
---
Labels: newbie  (was: )

> FairScheduler.initScheduler() contains a surprising unary plus
> --
>
> Key: YARN-7132
> URL: https://issues.apache.org/jira/browse/YARN-7132
> Project: Hadoop YARN
>  Issue Type: Improvement
>  Components: fairscheduler
>Affects Versions: 3.0.0-alpha4
>Reporter: Daniel Templeton
>Priority: Minor
>  Labels: newbie
>
> The method contains the following code:{code}
> LOG.warn(FairSchedulerConfiguration.UPDATE_INTERVAL_MS
> + " is invalid, so using default value "
> + +FairSchedulerConfiguration.DEFAULT_UPDATE_INTERVAL_MS
> + " ms instead");{code}
> Note the beginning of the third line.  One of those plusses should be deleted 
> so that no one else spends cycles trying to understand why it even compiles.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org