nathadfield commented on PR #67285: URL: https://github.com/apache/airflow/pull/67285#issuecomment-4678589441
@Lee-W one idea: rather than returning a tuple, we could pass the carried date into the resolver and keep the return type a plain `datetime | None`, e.g. `_resolve_partition_date(..., carried_partition_date=apdr.partition_date)`. Then the three exits are: anchors agree, return the anchor; nothing temporal contributed, return the carry; conflict or error, return `None`. The fallback branch at the call site disappears and the suppression rule lives in one place. Happy to push that if it works for you. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
