rmetzger commented on issue #9601: [FLINK-13894][web]Web Ui add log url for 
subtask of vertex
URL: https://github.com/apache/flink/pull/9601#issuecomment-543145071
 
 
   Thanks a lot for this pull request.
   I agree with Chesnay, that the initial approach of adding a "Logs" link next 
to the ID does not make sense conceptually.
   The current approach...
   
   
![image](https://user-images.githubusercontent.com/89049/67007220-30e5f800-f0e7-11e9-9080-d0b1b0bfc395.png)
   
   ... is also not good in my opinion, for the following reasons:
   - This view shows metrics per subtask ID, the "log" link is a little bit 
lost there
   - I don't understand the decision to use a lowercase "l". I would write 
"Log".
   - I'm not sure if it is intuitive for users to understand that this "Log" is 
the TaskManager log. As a user, I would expect a page that shows the log 
messages only from this specific SubTask, which is not true.
   
   I don't have much experience with UX design, and I don't have much time to 
really think this through (I would recommend you to do that), but one idea I 
had was to solve this similar to how Google does their UI:
   <img width="449" alt="Screenshot 2019-10-17 at 14 08 32" 
src="https://user-images.githubusercontent.com/89049/67007446-b2d62100-f0e7-11e9-8829-e0fedaa759df.png";>
   With that approach, we could have the three "..." at the end of the line, 
and then a dropdown that says: "View TaskManager logs"
   
   

----------------------------------------------------------------
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