GJL edited a comment on pull request #11731:
URL: https://github.com/apache/flink/pull/11731#issuecomment-620048458


   First of all, I am sorry that I did not see this problem earlier.
   
   > 1. split the taskmanager and jobmanager logs into different folders to 
avoid they mixed together in the standalone mode (which may bring break 
changes?).
   >
   > 2. add lastModified describe in FLINK-16863 and current:boolean data in 
the list item.
   
   It would mitigate the issue. However, the user would still see additional 
log files belonging to other processes. Ideally, only files owned by the 
respective TM/JM process are listed. Therefore, an extension of your idea that 
@tillrohrmann had is to put all logs belonging to a single process into a 
separate directory. However, for a user who wants to configure _custom_ log 
files, it could be difficult to predict the name of the log directory, which 
would render the whole feature useless.
   
   A (temporary) workaround could be to always show the main log and stdout 
file by default and make the log list a separate page. This would allow us to 
roll out the feature while not changing existing behavior.
   
   I think we are unable to make a decision on our own. How about we bring this 
issue up on the dev mailing list and see what others are saying (especially the 
developers that voted on FLIP-103 in the first place)?
   
   
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to