[GitHub] nlu90 commented on a change in pull request #2941: add tmaster physical plan endpoint

2018-06-28 Thread GitBox
nlu90 commented on a change in pull request #2941: add tmaster physical plan endpoint URL: https://github.com/apache/incubator-heron/pull/2941#discussion_r198898859 ## File path: heron/tmaster/src/cpp/manager/tcontroller.cpp ## @@ -262,6 +269,38 @@ void

[GitHub] yaoliclshlmch commented on a change in pull request #2941: add tmaster physical plan endpoint

2018-06-28 Thread GitBox
yaoliclshlmch commented on a change in pull request #2941: add tmaster physical plan endpoint URL: https://github.com/apache/incubator-heron/pull/2941#discussion_r198936686 ## File path: heron/tmaster/src/cpp/manager/tcontroller.cpp ## @@ -262,6 +269,38 @@ void

[GitHub] huijunw commented on issue #2927: [dhalion] null exception for previousCheckpoint

2018-06-28 Thread GitBox
huijunw commented on issue #2927: [dhalion] null exception for previousCheckpoint URL: https://github.com/apache/incubator-heron/issues/2927#issuecomment-400982016 Hi @avflor The healthmgr failed and quited in the first round policy execution, then restarted. There were no last 5

[GitHub] huijunw edited a comment on issue #2927: [dhalion] null exception for previousCheckpoint

2018-06-28 Thread GitBox
huijunw edited a comment on issue #2927: [dhalion] null exception for previousCheckpoint URL: https://github.com/apache/incubator-heron/issues/2927#issuecomment-400982016 Hi @avflor The healthmgr failed and quited in the first round policy execution, then restarted. There were no

[GitHub] yaoliclshlmch closed pull request #2942: [do not review] test travis ci

2018-06-28 Thread GitBox
yaoliclshlmch closed pull request #2942: [do not review] test travis ci URL: https://github.com/apache/incubator-heron/pull/2942 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