Ardagan edited a comment on pull request #11555:
URL: https://github.com/apache/beam/pull/11555#issuecomment-622141777


   Can be useful to add link instructions on how to access dashboards into PR 
description.
   [CWIKI](https://cwiki.apache.org/confluence/display/BEAM/Community+Metrics), 
section Grafana UI
   Summary:
   docker-compose build
   docker-compose up
   http://localhost:3000
   login with "admin":\<check password in docker-compose.yml\>
   
   I took a brief look at dashboards, here are my ideas:
   
   - I'd put graphs with similar arguments side-by-side, make graphs narrower. 
Goal is to see general sense of current status. Current layout is too sparse 
and it takes way too much time to scroll.
   - Set default dashboard interval to last month. If you feel month is not 
enough, choose value you feel more relevant.
   - Some space can be recovered by removing time scale.
   - I think that similar metrics can be put into single graph, ex: TextIO 1Gb 
[GCS, HDFS].
   - I would add a short description of what tests are executed. We all can 
access code, but it would be much more handy. This can be done either as a text 
field at the top of dashboard, or as a description hint on each of graphs, or 
both.
   
   These are my initial thoughts.
   
   R: @aaltay 
   Ahmet, I think you might be interested as well.
   


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


Reply via email to