[GitHub] incubator-gearpump issue #170: [GEARPUMP-291] Refactor TaskActor

2017-03-10 Thread codecov-io
Github user codecov-io commented on the issue:

https://github.com/apache/incubator-gearpump/pull/170
  
# 
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=h1)
 Report
> Merging 
[#170](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-gearpump/commit/aebc09a8deba2c68a0664f0035d9e68e5c4219a6?src=pr&el=desc)
 will **decrease** coverage by `0.29%`.
> The diff coverage is `78.26%`.



```diff
@@Coverage Diff@@
##   master #170 +/-   ##
=
- Coverage   71.91%   71.62%   -0.3% 
=
  Files 191  191 
  Lines6079 6086  +7 
  Branches  509  527 +18 
=
- Hits 4372 4359 -13 
- Misses   1707 1727 +20
```


--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=continue).
> **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute  (impact)`, `ø = not affected`, `? = missing 
data`
> Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=footer).
 Last update 
[aebc09a...38c8de1](https://codecov.io/gh/apache/incubator-gearpump/compare/aebc09a8deba2c68a0664f0035d9e68e5c4219a6...38c8de1b09d5d1b630ae7d1ebaf348abe4417cfa?src=pr&el=footer&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (GEARPUMP-291) TaskActor refactoring

2017-03-10 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-291:
-

Github user codecov-io commented on the issue:

https://github.com/apache/incubator-gearpump/pull/170
  
# 
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=h1)
 Report
> Merging 
[#170](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-gearpump/commit/aebc09a8deba2c68a0664f0035d9e68e5c4219a6?src=pr&el=desc)
 will **decrease** coverage by `0.29%`.
> The diff coverage is `78.26%`.



```diff
@@Coverage Diff@@
##   master #170 +/-   ##
=
- Coverage   71.91%   71.62%   -0.3% 
=
  Files 191  191 
  Lines6079 6086  +7 
  Branches  509  527 +18 
=
- Hits 4372 4359 -13 
- Misses   1707 1727 +20
```


--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=continue).
> **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
> Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/170?src=pr&el=footer).
 Last update 
[aebc09a...38c8de1](https://codecov.io/gh/apache/incubator-gearpump/compare/aebc09a8deba2c68a0664f0035d9e68e5c4219a6...38c8de1b09d5d1b630ae7d1ebaf348abe4417cfa?src=pr&el=footer&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


> TaskActor refactoring
> -
>
> Key: GEARPUMP-291
> URL: https://issues.apache.org/jira/browse/GEARPUMP-291
> Project: Apache Gearpump
>  Issue Type: Improvement
>  Components: streaming
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Manu Zhang
>
> To fix warnings and styles, avoid tricky codes, and decompose "big" function 
> into "smaller" modules. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] incubator-gearpump pull request #169: [GEARPUMP-285] fix false alarm of shut...

2017-03-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-gearpump pull request #170: [GEARPUMP-291] Refactor TaskActor

2017-03-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-gearpump pull request #170: [GEARPUMP-291] Refactor TaskActor

2017-03-10 Thread manuzhang
GitHub user manuzhang opened a pull request:

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

[GEARPUMP-291] Refactor TaskActor

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 refactor_task

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

https://github.com/apache/incubator-gearpump/pull/170.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 #170


commit 991b4fa4c73dc3cdc53e4f495871cc0dd0764fdc
Author: manuzhang 
Date:   2017-03-10T07:48:47Z

[GEARPUMP-291] Refactor TaskActor




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-gearpump issue #169: [GEARPUMP-285] fix false alarm of shutting do...

2017-03-10 Thread manuzhang
Github user manuzhang commented on the issue:

https://github.com/apache/incubator-gearpump/pull/169
  
+1, merging


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-gearpump issue #170: [GEARPUMP-291] Refactor TaskActor

2017-03-10 Thread manuzhang
Github user manuzhang commented on the issue:

https://github.com/apache/incubator-gearpump/pull/170
  
R: @huafengw test passes 
https://travis-ci.org/manuzhang/incubator-gearpump/builds/209659694


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-gearpump issue #170: [GEARPUMP-291] Refactor TaskActor

2017-03-10 Thread huafengw
Github user huafengw commented on the issue:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (GEARPUMP-291) TaskActor refactoring

2017-03-10 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-291:
-

Github user asfgit closed the pull request at:

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


> TaskActor refactoring
> -
>
> Key: GEARPUMP-291
> URL: https://issues.apache.org/jira/browse/GEARPUMP-291
> Project: Apache Gearpump
>  Issue Type: Improvement
>  Components: streaming
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Manu Zhang
>
> To fix warnings and styles, avoid tricky codes, and decompose "big" function 
> into "smaller" modules. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (GEARPUMP-291) TaskActor refactoring

2017-03-10 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-291:
-

Github user huafengw commented on the issue:

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


> TaskActor refactoring
> -
>
> Key: GEARPUMP-291
> URL: https://issues.apache.org/jira/browse/GEARPUMP-291
> Project: Apache Gearpump
>  Issue Type: Improvement
>  Components: streaming
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Manu Zhang
>
> To fix warnings and styles, avoid tricky codes, and decompose "big" function 
> into "smaller" modules. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (GEARPUMP-291) TaskActor refactoring

2017-03-10 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-291:
-

Github user manuzhang commented on the issue:

https://github.com/apache/incubator-gearpump/pull/170
  
R: @huafengw test passes 
https://travis-ci.org/manuzhang/incubator-gearpump/builds/209659694


> TaskActor refactoring
> -
>
> Key: GEARPUMP-291
> URL: https://issues.apache.org/jira/browse/GEARPUMP-291
> Project: Apache Gearpump
>  Issue Type: Improvement
>  Components: streaming
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Manu Zhang
>
> To fix warnings and styles, avoid tricky codes, and decompose "big" function 
> into "smaller" modules. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Resolved] (GEARPUMP-285) AppManager shut down executor time out

2017-03-10 Thread Manu Zhang (JIRA)

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

Manu Zhang resolved GEARPUMP-285.
-
   Resolution: Fixed
Fix Version/s: 0.8.3

Issue resolved by pull request 169
[https://github.com/apache/incubator-gearpump/pull/169]

> AppManager shut down executor time out
> --
>
> Key: GEARPUMP-285
> URL: https://issues.apache.org/jira/browse/GEARPUMP-285
> Project: Apache Gearpump
>  Issue Type: Bug
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Huafeng Wang
>Priority: Minor
> Fix For: 0.8.3
>
>
> When running Beam tests for about 10 seconds with {{EmbeddedCluster}}, I saw 
> the "Shut down executor time out" error log which should be trigger only 
> after 30 seconds. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (GEARPUMP-285) AppManager shut down executor time out

2017-03-10 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GEARPUMP-285:
-

Github user asfgit closed the pull request at:

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


> AppManager shut down executor time out
> --
>
> Key: GEARPUMP-285
> URL: https://issues.apache.org/jira/browse/GEARPUMP-285
> Project: Apache Gearpump
>  Issue Type: Bug
>Affects Versions: 0.8.2
>Reporter: Manu Zhang
>Assignee: Huafeng Wang
>Priority: Minor
> Fix For: 0.8.3
>
>
> When running Beam tests for about 10 seconds with {{EmbeddedCluster}}, I saw 
> the "Shut down executor time out" error log which should be trigger only 
> after 30 seconds. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)