Re: Stateful Updates

2018-03-30 Thread Karthik Ramasamy
Thanks Tom. It will be a great addition. Unable to access the doc. Could you give permission? Sent from my iPhone > On Mar 30, 2018, at 6:08 PM, Thomas Cooper wrote: > > Hi all, > > After chatting to the Heron Team at Twitter and Microsoft I have written up > a

[GitHub] joshfischer1108 opened a new pull request #2830: removing abstract requirement

2018-03-30 Thread GitBox
joshfischer1108 opened a new pull request #2830: removing abstract requirement URL: https://github.com/apache/incubator-heron/pull/2830 @kramasamy @nwangtw This is fix for ECO custom grouping definitions. This is an automated

[GitHub] ashvina commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
ashvina commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178407367 ## File path: heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/BackPressureDetector.java ##

[GitHub] huijunwu commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
huijunwu commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178395687 ## File path: heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/BackPressureDetector.java ##

[GitHub] huijunwu commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
huijunwu commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178394376 ## File path: heron/executor/src/python/heron_executor.py ## @@ -498,7 +498,7 @@ def

[GitHub] huijunwu commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
huijunwu commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178399304 ## File path: heron/healthmgr/src/java/com/twitter/heron/healthmgr/diagnosers/SlowInstanceDiagnoser.java

[GitHub] huijunwu commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
huijunwu commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178397892 ## File path: heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/GrowingWaitQueueDetector.java

[GitHub] huijunwu commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
huijunwu commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178398628 ## File path: heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/SkewDetector.java ## @@

[GitHub] ashvina commented on a change in pull request #2821: Update Dhalion dependency version

2018-03-30 Thread GitBox
ashvina commented on a change in pull request #2821: Update Dhalion dependency version URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178357054 ## File path: heron/executor/src/python/heron_executor.py ## @@ -498,7 +498,7 @@ def

Re: [MENTORS] Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Ning Wang
For 2, 0.17.7 was released in late Feb I believe and there have been new features/fixes since then. On Fri, Mar 30, 2018 at 10:31 AM, Dave Fisher wrote: > Hi - > > Adding the tag that Taylor mentioned to help signal when the project has > questions. > > While I on the

Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Ashvin A
+1 On Fri, Mar 30, 2018 at 10:22 AM, Eren Avsarogullari < erenavsarogull...@gmail.com> wrote: > SGTM > > On 30 March 2018 at 18:19, Ning Wang wrote: > > > SGTM > > > > On Fri, Mar 30, 2018 at 10:05 AM, Karthik Ramasamy > > wrote: > > > > > All - > > >

Re: CI job for heron

2018-03-30 Thread Ning Wang
Got it. Thanks. I will take a look. The previous Travis service is a Twitter paid service I think, so it is time to consider our options. On Fri, Mar 30, 2018 at 10:36 AM, Dave Fisher wrote: > Hi - > > > On Mar 29, 2018, at 11:37 PM, Ning Wang

Re: CI job for heron

2018-03-30 Thread Dave Fisher
Hi - > On Mar 29, 2018, at 11:37 PM, Ning Wang wrote: > > Hi, Dave, > > It seems that we need to update heron CI job for unit tests and integration > tests. We used Travis previously. I am wondering if there is a CI engine in > Apache infra? Or is there any suggestion?

[MENTORS] Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Dave Fisher
Hi - Adding the tag that Taylor mentioned to help signal when the project has questions. While I on the IPMC I am not one of your mentors. A few questions to answer about this non-Apache release. (1) where will the resulting artifacts be published? (2) How is it different from the prior

Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Josh Fischer
+1 -Josh On Fri, Mar 30, 2018 at 12:22 PM Eren Avsarogullari < erenavsarogull...@gmail.com> wrote: > SGTM > > On 30 March 2018 at 18:19, Ning Wang wrote: > > > SGTM > > > > On Fri, Mar 30, 2018 at 10:05 AM, Karthik Ramasamy > > wrote: > > > > > All - >

Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Eren Avsarogullari
SGTM On 30 March 2018 at 18:19, Ning Wang wrote: > SGTM > > On Fri, Mar 30, 2018 at 10:05 AM, Karthik Ramasamy > wrote: > > > All - > > > > Since we have been planning to release 0.17.8 till last week, I would > > suggest that we can go ahead with this

Re: Heron Non Apache Release 0.17.8

2018-03-30 Thread Ning Wang
SGTM On Fri, Mar 30, 2018 at 10:05 AM, Karthik Ramasamy wrote: > All - > > Since we have been planning to release 0.17.8 till last week, I would > suggest that we can go ahead with this release. However this release will > be a non Apache release since there are several task

Heron Non Apache Release 0.17.8

2018-03-30 Thread Karthik Ramasamy
All - Since we have been planning to release 0.17.8 till last week, I would suggest that we can go ahead with this release. However this release will be a non Apache release since there are several task items that needs to be done before making an Apache release. I would suggest the following -

[GitHub] nwangtw opened a new issue #2828: Rename package name from com.twitter.heron to com.apache.com

2018-03-30 Thread GitBox
nwangtw opened a new issue #2828: Rename package name from com.twitter.heron to com.apache.com URL: https://github.com/apache/incubator-heron/issues/2828 This is an automated message from the Apache Git Service. To respond

[GitHub] nwangtw opened a new issue #2829: Update license to apache incubator

2018-03-30 Thread GitBox
nwangtw opened a new issue #2829: Update license to apache incubator URL: https://github.com/apache/incubator-heron/issues/2829 This is an automated message from the Apache Git Service. To respond to the message, please log

CI job for heron

2018-03-30 Thread Ning Wang
Hi, Dave, It seems that we need to update heron CI job for unit tests and integration tests. We used Travis previously. I am wondering if there is a CI engine in Apache infra? Or is there any suggestion? Thanks in advance.

[GitHub] nwangtw closed pull request #2822: Add rate limit change log in stream manager

2018-03-30 Thread GitBox
nwangtw closed pull request #2822: Add rate limit change log in stream manager URL: https://github.com/apache/incubator-heron/pull/2822 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] nwangtw opened a new issue #2827: Fix CI job for compiling/unit tests/integration tests

2018-03-30 Thread GitBox
nwangtw opened a new issue #2827: Fix CI job for compiling/unit tests/integration tests URL: https://github.com/apache/incubator-heron/issues/2827 After migrating to apache, the old travis job stops working. Need to figure out the new way to run CI.