Taragolis commented on PR #29395:
URL: https://github.com/apache/airflow/pull/29395#issuecomment-1420022737

   > I think better to combine it - it will make it more cherry-pickable. I 
think this one is a candidate to 2.5.2 because of this 'side-effect' behaviour 
for jsonschema/parameters
   
   I will check my branch with changes from this branch In the morning or 
afternoon.
   
   One thing what I think we should decide, we have two options with Param 
date-time:
   1. As it implemented right now in 
https://github.com/apache/airflow/pull/29374, convert values iso8601 -> rfc3339 
and naive -> awaire
   2. Just check is it valid iso8601, in case if rfc3339 validation failed, but 
do not change actual values and only warn users


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