Daniel Standish
Daniel Standish
> @dstandish and team - I see this issue was auto-closed and now reopened. My team is struggling with the same issue, so we'd be happy to help/test in any...
> @dstandish - To clarify, is this PR ready to test as-is? Right. In short what this does is, now when trigger times out we raise TaskDeferralTimeout instead of the...
@robg-eb were you able to test this locally?
> @dstandish I had tested this out on Jan 29 2024 and left the feedback in the comment above - [#33718 (comment)](https://github.com/apache/airflow/pull/33718#issuecomment-1914788890). I see there's now a new commit as...
> We need to keep this active and gain consistency. Ok @nathadfield I just rebased. I pinged @hussein-awala to pick the comment thread back up. If you could review the...
> However, we found another inconsistency between sync and async sensors here. In the case where an interruption does happen and a retry is needed, it seems with this implementation,...
@robg-eb looked at your comment > However, we found another inconsistency between sync and async sensors here. In the case where an interruption does happen and a retry is needed,...
ship it @o-nikolas
let me highlight some complexities here. when reading logs for a running tasks, the logs could be various locations: * locally * blob storage * served from triggerer * served...
> @dstandish I believe this would only apply to completed tasks? I don't think running tasks serve logs from blob storage i don't see anything here that suggests this conditionally...