feng-tao commented on issue #4396: [AIRFLOW-3585] - Add edges to database
URL: https://github.com/apache/airflow/pull/4396#issuecomment-465237291
 
 
   thanks @astahlman for the extensive testing.
   
   @ffinfo , I feel at this point,  there seems to be more work than we think. 
The change needs some UX improvement as well IMO. e,g: I don't personally like 
adding the `read from dag file` button in every graph view. And I don't want 
@astahlman  to keep testing your branch and found more issues.
   
   If you want to keep pursuing this pr, I request you should conduct the test 
with your branch on various scheduler(local, celery, backfill or even k8s) and 
post us the result with gif and picture on how it goes. 
   
   Once you feel comfortable with your branch, I would suggest you send an 
email to your airflow-dev AIF discussion email thread and get the feedback. 
Once the community aligns, we could do some more tests and see how it goes. 
WDYT.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to