[GitHub] nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196256952 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/common/

[GitHub] huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196254210 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/policy/

[incubator-heron] branch huijunw/togglepolicy updated: fix neng comment

2018-06-18 Thread huijun
This is an automated email from the ASF dual-hosted git repository. huijun pushed a commit to branch huijunw/togglepolicy in repository https://gitbox.apache.org/repos/asf/incubator-heron.git The following commit(s) were added to refs/heads/huijunw/togglepolicy by this push: new f24a2b6 f

[GitHub] huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196253044 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/common/

[GitHub] huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
huijunw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196252303 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/policy/

[GitHub] srkukarni commented on a change in pull request #2891: Refactor StatefulStorage API

2018-06-18 Thread GitBox
srkukarni commented on a change in pull request #2891: Refactor StatefulStorage API URL: https://github.com/apache/incubator-heron/pull/2891#discussion_r196244013 ## File path: heron/proto/ckptmgr.proto ## @@ -171,7 +175,23 @@ message CleanStatefulCheckpointResponse { r

[GitHub] srkukarni commented on a change in pull request #2891: Refactor StatefulStorage API

2018-06-18 Thread GitBox
srkukarni commented on a change in pull request #2891: Refactor StatefulStorage API URL: https://github.com/apache/incubator-heron/pull/2891#discussion_r196243889 ## File path: heron/proto/ckptmgr.proto ## @@ -171,7 +175,23 @@ message CleanStatefulCheckpointResponse { r

[GitHub] nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196239712 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/common/

[GitHub] nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
nwangtw commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196243284 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/policy/

[GitHub] nlu90 commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
nlu90 commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196236723 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/policy/To

[GitHub] nlu90 commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
nlu90 commented on a change in pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923#discussion_r196236440 ## File path: heron/healthmgr/src/java/org/apache/heron/healthmgr/policy/To

[GitHub] huijunwu opened a new pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy

2018-06-18 Thread GitBox
huijunwu opened a new pull request #2923: [healthmgr] Enable runtime toggle for AutoRestartBackpressureContainerPolicy URL: https://github.com/apache/incubator-heron/pull/2923 Heron supports config update at runtime. This PR enforces toggle config for AutoRestartBackpressureContainerPolicy

[incubator-heron] branch huijunw/togglepolicy updated: clean comment

2018-06-18 Thread huijun
This is an automated email from the ASF dual-hosted git repository. huijun pushed a commit to branch huijunw/togglepolicy in repository https://gitbox.apache.org/repos/asf/incubator-heron.git The following commit(s) were added to refs/heads/huijunw/togglepolicy by this push: new e28ea14 c

[incubator-heron] 01/02: enable runtime config for autobackpressure policy

2018-06-18 Thread huijun
This is an automated email from the ASF dual-hosted git repository. huijun pushed a commit to branch huijunw/togglepolicy in repository https://gitbox.apache.org/repos/asf/incubator-heron.git commit 1fce9aff12eea2a3d545b71af3b73a5c5d09519f Author: Huijun Wu AuthorDate: Mon Jun 18 14:25:34 2018 -

[incubator-heron] branch huijunw/togglepolicy created (now 39931fa)

2018-06-18 Thread huijun
This is an automated email from the ASF dual-hosted git repository. huijun pushed a change to branch huijunw/togglepolicy in repository https://gitbox.apache.org/repos/asf/incubator-heron.git. at 39931fa update comment This branch includes the following new commits: new 1fce9af en

[incubator-heron] 02/02: update comment

2018-06-18 Thread huijun
This is an automated email from the ASF dual-hosted git repository. huijun pushed a commit to branch huijunw/togglepolicy in repository https://gitbox.apache.org/repos/asf/incubator-heron.git commit 39931faded90e76e5661ab3dd7ab31a79cbe93ea Author: Huijun Wu AuthorDate: Mon Jun 18 14:30:39 2018 -

[GitHub] Code0x58 closed pull request #2915: [WIP] Use six to make heronpy tests pass in python3

2018-06-18 Thread GitBox
Code0x58 closed pull request #2915: [WIP] Use six to make heronpy tests pass in python3 URL: https://github.com/apache/incubator-heron/pull/2915 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

[incubator-heron] branch revert-2921-add-jdk-to-bazel_configure deleted (was c50506e)

2018-06-18 Thread karthikz
This is an automated email from the ASF dual-hosted git repository. karthikz pushed a change to branch revert-2921-add-jdk-to-bazel_configure in repository https://gitbox.apache.org/repos/asf/incubator-heron.git. was c50506e Revert "Add JDK/JAVA_HOME to bazel_configure.py (#2921)" This ch

[incubator-heron] branch revert-2921-add-jdk-to-bazel_configure created (now c50506e)

2018-06-18 Thread karthikz
This is an automated email from the ASF dual-hosted git repository. karthikz pushed a change to branch revert-2921-add-jdk-to-bazel_configure in repository https://gitbox.apache.org/repos/asf/incubator-heron.git. at c50506e Revert "Add JDK/JAVA_HOME to bazel_configure.py (#2921)" This br

[incubator-heron] 01/01: Revert "Add JDK/JAVA_HOME to bazel_configure.py (#2921)"

2018-06-18 Thread karthikz
This is an automated email from the ASF dual-hosted git repository. karthikz pushed a commit to branch revert-2921-add-jdk-to-bazel_configure in repository https://gitbox.apache.org/repos/asf/incubator-heron.git commit c50506ef656f4dbac1e0347b9a39e457d73254de Author: Karthik Ramasamy AuthorDate:

[incubator-heron] branch master updated: Add JDK/JAVA_HOME to bazel_configure.py (#2921)

2018-06-18 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 b3522d2 Add JDK/JAVA_HOME to bazel_c

[GitHub] kramasamy closed pull request #2921: Add JDK/JAVA_HOME to bazel_configure.py

2018-06-18 Thread GitBox
kramasamy closed pull request #2921: Add JDK/JAVA_HOME to bazel_configure.py URL: https://github.com/apache/incubator-heron/pull/2921 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 fo

[GitHub] kramasamy commented on issue #2921: Add JDK/JAVA_HOME to bazel_configure.py

2018-06-18 Thread GitBox
kramasamy commented on issue #2921: Add JDK/JAVA_HOME to bazel_configure.py URL: https://github.com/apache/incubator-heron/pull/2921#issuecomment-398048450 @Code0x58 - there is no need to run bazel configure anymore. It can be used to check whether all the requirements are present. ---

[incubator-heron] branch master updated: Fix integer types to avoid warnings (#2922)

2018-06-18 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 81bb28d Fix integer types to avoid w

[GitHub] kramasamy closed pull request #2922: Fix integer types to avoid warnings

2018-06-18 Thread GitBox
kramasamy closed pull request #2922: Fix integer types to avoid warnings URL: https://github.com/apache/incubator-heron/pull/2922 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 foreig

[GitHub] kramasamy commented on issue #2922: Fix integer types to avoid warnings

2018-06-18 Thread GitBox
kramasamy commented on issue #2922: Fix integer types to avoid warnings URL: https://github.com/apache/incubator-heron/pull/2922#issuecomment-398047591 @Code0x58 - thanks for the cleanup. This is an automated message from the

[GitHub] kramasamy closed pull request #2920: Correct case of acronyms

2018-06-18 Thread GitBox
kramasamy closed pull request #2920: Correct case of acronyms URL: https://github.com/apache/incubator-heron/pull/2920 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 requ

[incubator-heron] branch master updated: Correct case of acronyms (#2920)

2018-06-18 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 58078e7 Correct case of acronyms (#2

[GitHub] Code0x58 opened a new pull request #2922: Fix integer types to avoid warnings

2018-06-18 Thread GitBox
Code0x58 opened a new pull request #2922: Fix integer types to avoid warnings URL: https://github.com/apache/incubator-heron/pull/2922 The output was a bit noisy because of signed/unsigned comparisons so I cleaned them up. Th

[GitHub] Code0x58 opened a new pull request #2921: Add JDK/JAVA_HOME to bazel_configure.py

2018-06-18 Thread GitBox
Code0x58 opened a new pull request #2921: Add JDK/JAVA_HOME to bazel_configure.py URL: https://github.com/apache/incubator-heron/pull/2921 I found that `jar` wasn't available when I did `bazel test heron/...`, so I thought it would be good to check for the presence of a JDK in configure.

[GitHub] Code0x58 commented on issue #2920: Correct dag to DAG

2018-06-18 Thread GitBox
Code0x58 commented on issue #2920: Correct dag to DAG URL: https://github.com/apache/incubator-heron/pull/2920#issuecomment-397961335 @kramasamy yep, I think it's good to go now This is an automated message from the Apache Git