vthinkxie commented on issue #10228: [FLINK-14816] Add thread dump feature for 
taskmanager
URL: https://github.com/apache/flink/pull/10228#issuecomment-613775212
 
 
   > Thanks a lot for creating this PR @lamber-ken and sorry that it took so 
long to get back to you. I've given your PR a round and it looks already very 
good. I had few comments. In particular, it would be good to add a test for 
this feature. Moreover, it would be awesome if you could rebase this PR onto 
the latest master because it has diverged a bit.
   > 
   > @vthinkxie could you help reviewing the web ui changes? I can only try it 
out.
   
   Hi @lamber-ken , the UI of TM logs has been changed according to 
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=147427143
   could you rebase the frontend part?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to