[GitHub] [incubator-streampark] RocMarshal commented on pull request #2675: [ISSUE-2498][Feature] [SubTask] The cluster supports remote and yarn session heartbeat monitoring

["RocMarshal (via GitHub)" <gi...@apache.org>]

RocMarshal commented on PR #2675:
URL: https://github.com/apache/incubator-streampark/pull/2675#issuecomment-1518639081

   > Hi @RocMarshal ,This is my resubmitted PR for #2541. This PR has modified some suggestions for #2541.
   > 
   > For the issue of [#2541 (comment)](https://github.com/apache/incubator-streampark/pull/2541#discussion_r1162910882), I have thought about it for a moment. In fact, it will not have an impact on obtaining the cluster status because the cluster failed to obtain the corresponding status information from` jobManagerUrl` and will go back to Yarn.
   > 
   > Looking forward to your next review.
   
   Thank you @xujiangfeng001  very much for your update. It seems closer to the ideal state.
   
   Due to my personal reasons, the review may be conducted later.
   Other reviewers CC @wolfboys @macksonmu  can also participate in this MR review.
   If you have time on your end, you can try to continue to advance the design and discussion of other subtasks of https://github.com/apache/incubator-streampark/issues/2425
   which will shorten some of the connection  time~
   Best Roc.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org