You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2018/09/17 09:39:38 UTC

[GitHub] robin-miller-ow commented on issue #1910: [AIRFLOW-659] Automatic Refresh on DAG Graph View

robin-miller-ow commented on issue #1910: [AIRFLOW-659] Automatic Refresh on DAG Graph View
URL: https://github.com/apache/incubator-airflow/pull/1910#issuecomment-421945706
 
 
   Hi @r39132 
   
   I don't currently have time to put towards this unfortunately. I'm happy if you or anybody else wanted to pick up what I've done here and add to it before it's merged in.
   
   If you are working further along these lines, one thing that might be of note: we've been using this as it is here in our deployment and have concluded that it would be worth disabling the loading spinner that occurs when the graph view is refreshed automatically, as it tends to get in the way.
   
   Regarding your questions:
   
   1. I see no reason why this could not be done, but I don't believe any of the other views have a corresponding refresh capability built in at the moment, so this would need to be created as part of this.
   1. Again, I see no reason not to do this. As mentioned above, I'm not likely to get time to work on this any time soon, but I'd be inclined to view that as an enhancement to this functionality that could be added later - turning the configuration value into a default at that point.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services