Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/3645
Thank you for reviews! @tillrohrmann
---
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
ena
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3645
Changes look good to me. Thanks for the contribution @zhijiangW. I will
rebase this PR and upon receiving green light from Travis I will merge it.
---
If your project is set up for it, you can
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/3645
@tillrohrmann , just a kind reminder for this last heartbeat PR.
---
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 doe
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/3645
@tillrohrmann , I have submitted the heartbeats between `JobManager` and
`ResourceManager`, wish your reviews! Sorry for a little delay because of our
team outing last week.
---
If your project