[jira] [Commented] (GEARPUMP-369) "executor-num" is overridden by the default value for arg "executors"(1)

2018-04-05 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-369:
-

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-gearpump/pull/242


> "executor-num" is overridden by the default value for arg "executors"(1) 
> -
>
> Key: GEARPUMP-369
> URL: https://issues.apache.org/jira/browse/GEARPUMP-369
> Project: Apache Gearpump
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.8.4, 0.8.5
>Reporter: Hashan Gayasri Udugahapattuwa
>Priority: Minor
>
> org/apache/gearpump/cluster/main/AppSubmitter.scala:62
> System.setProperty(Constants.APPLICATION_EXECUTOR_NUMBER, 
> config.getInt("executors").toString)
>  
> replaces the value set by system properties (gear.conf) by the default 
> value(1) even when no value is passed at as an argument. The expected 
> behaviour is to only override the value when a value is given explicitly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (GEARPUMP-369) "executor-num" is overridden by the default value for arg "executors"(1)

2018-04-05 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-369:
-

Github user huafengw commented on the issue:

https://github.com/apache/incubator-gearpump/pull/242
  
+1


> "executor-num" is overridden by the default value for arg "executors"(1) 
> -
>
> Key: GEARPUMP-369
> URL: https://issues.apache.org/jira/browse/GEARPUMP-369
> Project: Apache Gearpump
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.8.4, 0.8.5
>Reporter: Hashan Gayasri Udugahapattuwa
>Priority: Minor
>
> org/apache/gearpump/cluster/main/AppSubmitter.scala:62
> System.setProperty(Constants.APPLICATION_EXECUTOR_NUMBER, 
> config.getInt("executors").toString)
>  
> replaces the value set by system properties (gear.conf) by the default 
> value(1) even when no value is passed at as an argument. The expected 
> behaviour is to only override the value when a value is given explicitly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (GEARPUMP-369) "executor-num" is overridden by the default value for arg "executors"(1)

2018-04-05 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-369:
-

Github user codecov-io commented on the issue:

https://github.com/apache/incubator-gearpump/pull/242
  
# 
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/242?src=pr&el=h1)
 Report
> Merging 
[#242](https://codecov.io/gh/apache/incubator-gearpump/pull/242?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-gearpump/commit/8aac07b86d4b90e4f5118f3204610d3510f93b02?src=pr&el=desc)
 will **increase** coverage by `0.09%`.
> The diff coverage is `50%`.



```diff
@@Coverage Diff @@
##   master #242  +/-   ##
==
+ Coverage   68.94%   69.03%   +0.09% 
==
  Files 191  191  
  Lines6128 6130   +2 
  Branches  542  540   -2 
==
+ Hits 4225 4232   +7 
+ Misses   1903 1898   -5
```




> "executor-num" is overridden by the default value for arg "executors"(1) 
> -
>
> Key: GEARPUMP-369
> URL: https://issues.apache.org/jira/browse/GEARPUMP-369
> Project: Apache Gearpump
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.8.4, 0.8.5
>Reporter: Hashan Gayasri Udugahapattuwa
>Priority: Minor
>
> org/apache/gearpump/cluster/main/AppSubmitter.scala:62
> System.setProperty(Constants.APPLICATION_EXECUTOR_NUMBER, 
> config.getInt("executors").toString)
>  
> replaces the value set by system properties (gear.conf) by the default 
> value(1) even when no value is passed at as an argument. The expected 
> behaviour is to only override the value when a value is given explicitly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (GEARPUMP-369) "executor-num" is overridden by the default value for arg "executors"(1)

2018-04-05 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-369:
-

Github user manuzhang commented on the issue:

https://github.com/apache/incubator-gearpump/pull/242
  
R: @huafengw 


> "executor-num" is overridden by the default value for arg "executors"(1) 
> -
>
> Key: GEARPUMP-369
> URL: https://issues.apache.org/jira/browse/GEARPUMP-369
> Project: Apache Gearpump
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.8.4, 0.8.5
>Reporter: Hashan Gayasri Udugahapattuwa
>Priority: Minor
>
> org/apache/gearpump/cluster/main/AppSubmitter.scala:62
> System.setProperty(Constants.APPLICATION_EXECUTOR_NUMBER, 
> config.getInt("executors").toString)
>  
> replaces the value set by system properties (gear.conf) by the default 
> value(1) even when no value is passed at as an argument. The expected 
> behaviour is to only override the value when a value is given explicitly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (GEARPUMP-369) "executor-num" is overridden by the default value for arg "executors"(1)

2018-04-05 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-369:
-

GitHub user manuzhang opened a pull request:

https://github.com/apache/incubator-gearpump/pull/242

[GEARPUMP-369] Fix configuring number of executors

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the commit message is formatted like:
   `[GEARPUMP-] Meaningful description of pull request` 
 - [ ] Make sure tests pass via `sbt clean test`.
 - [ ] Make sure old documentation affected by the pull request has been 
updated and new documentation added for new functionality. 



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/manuzhang/incubator-gearpump 
fix_config_executor_num

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-gearpump/pull/242.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #242


commit 669f4e1ea4669d98fe34a56900bd41a03918787d
Author: manuzhang 
Date:   2018-04-05T10:04:35Z

[GEARPUMP-369] Fix configuring number of executors




> "executor-num" is overridden by the default value for arg "executors"(1) 
> -
>
> Key: GEARPUMP-369
> URL: https://issues.apache.org/jira/browse/GEARPUMP-369
> Project: Apache Gearpump
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.8.4, 0.8.5
>Reporter: Hashan Gayasri Udugahapattuwa
>Priority: Minor
>
> org/apache/gearpump/cluster/main/AppSubmitter.scala:62
> System.setProperty(Constants.APPLICATION_EXECUTOR_NUMBER, 
> config.getInt("executors").toString)
>  
> replaces the value set by system properties (gear.conf) by the default 
> value(1) even when no value is passed at as an argument. The expected 
> behaviour is to only override the value when a value is given explicitly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)