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 2021/11/10 17:24:35 UTC

[GitHub] [airflow] o-nikolas edited a comment on pull request #19410: Remove inaccurate execution date from triggered dag extra link

o-nikolas edited a comment on pull request #19410:
URL: https://github.com/apache/airflow/pull/19410#issuecomment-965562760


   > > This fix doesn't look right to me -- by not including the execution date it will link to the latest run instead.
   > 
   > @ashb The issue working to be mitigated here is that the execution date parameter wasn't being used properly at all anyway and was linking to a poorly filtered view of the DAG, so mitigating it at least allows the user to view the latest and filter easily to the target.
   
   Hey @ashb,
   @ljades summarized it correctly. I know the original issue is quite long, but if you read that, it that will provide a lot of context so that this makes sense.
   
   Basically a "real fix" is quite tricky and I wasn't getting much feedback in the ticket, so as a stop gap this PR will at least remove the filter so that you see all runs from current run back which seemed like a decent default for the time being.


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org