Lee-W commented on issue #63442:
URL: https://github.com/apache/airflow/issues/63442#issuecomment-4046729209

   > My intuition is, instead of parsing the upstream partition key, the mapper 
should probably use the partition date directly. Is this even viable? I feel it 
needs to be.
   
   I thought of it, but I think it's better not to do this. Those mappers are 
meant to transform the datetime into str. This would confuse the logic when we 
want to support non-time-based ones.
   
   ---
   
   I don't think this is a bug. This is also mentioned in the doc PR 
https://github.com/apache/airflow/pull/63262/changes#diff-7ad271be3839a777f6889b0daf1571737fc3d15fec83befbd5da36d1d9de2fc9R525-R532
 , but I do have a UX improvement. https://github.com/apache/airflow/pull/62509
   
   
   
   


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

Reply via email to