[GitHub] yaoliclshlmch commented on issue #2956: add integration test for instance state in stateful processing

2018-07-12 Thread GitBox
yaoliclshlmch commented on issue #2956: add integration test for instance state in stateful processing URL: https://github.com/apache/incubator-heron/pull/2956#issuecomment-404670944 @huijunwu _"The IdentityBolt ABSpout IntegrationTopologyTestBolt IntegrationTopologyTestSpout look

[GitHub] yaoliclshlmch commented on a change in pull request #2956: add integration test for instance state in stateful processing

2018-07-12 Thread GitBox
yaoliclshlmch commented on a change in pull request #2956: add integration test for instance state in stateful processing URL: https://github.com/apache/incubator-heron/pull/2956#discussion_r202197020 ## File path:

[GitHub] yaoliclshlmch commented on a change in pull request #2956: add integration test for instance state in stateful processing

2018-07-12 Thread GitBox
yaoliclshlmch commented on a change in pull request #2956: add integration test for instance state in stateful processing URL: https://github.com/apache/incubator-heron/pull/2956#discussion_r202196585 ## File path:

[GitHub] kramasamy commented on issue #2950: Add signal handling to stream manager

2018-07-12 Thread GitBox
kramasamy commented on issue #2950: Add signal handling to stream manager URL: https://github.com/apache/incubator-heron/pull/2950#issuecomment-404577476 @nwangtw - can you please check if this is ok to merge it? @Glorfischi - have you fixed all the feedback.

[GitHub] kramasamy closed pull request #2960: joshfischer/update k8s docs

2018-07-12 Thread GitBox
kramasamy closed pull request #2960: joshfischer/update k8s docs URL: https://github.com/apache/incubator-heron/pull/2960 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 foreign pull

[incubator-heron] branch master updated: joshfischer/update k8s docs (#2960)

2018-07-12 Thread karthikz
This is an automated email from the ASF dual-hosted git repository. karthikz pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-heron.git The following commit(s) were added to refs/heads/master by this push: new aca05de joshfischer/update k8s docs

[GitHub] joshfischer1108 commented on issue #2960: joshfischer/update k8s docs

2018-07-12 Thread GitBox
joshfischer1108 commented on issue #2960: joshfischer/update k8s docs URL: https://github.com/apache/incubator-heron/pull/2960#issuecomment-404514502 Also found a file at `heron/deploy/kubernetes/helm/templates/NOTES.txt` that needed some changes. Not sure if that's used to dump to the

[GitHub] joshfischer1108 commented on issue #2960: joshfischer/update k8s docs

2018-07-12 Thread GitBox
joshfischer1108 commented on issue #2960: joshfischer/update k8s docs URL: https://github.com/apache/incubator-heron/pull/2960#issuecomment-404484659 @cckellogg Yes we should and glad you mentioned them. It totally slipped my mind. Just pushed them up.

[GitHub] nlu90 opened a new pull request #2962: [WIP] Enable instances and ckptmgr transferring state via local disk

2018-07-12 Thread GitBox
nlu90 opened a new pull request #2962: [WIP] Enable instances and ckptmgr transferring state via local disk URL: https://github.com/apache/incubator-heron/pull/2962 During our test, we found that if a state is too large (>~64MB), the original checkpointing mechanism fails. The actual