jason810496 commented on PR #61532: URL: https://github.com/apache/airflow/pull/61532#issuecomment-3875794596
> format: uuid is non-breaking for clients: OpenAPI's format is a hint. The base type is still string. Well-behaved client generators treat format: uuid as validation/documentation, not as a different wire type. Oh, indeed, I oversight the generated yaml spec, the type is still string instead of uuid, so I think it still compatible for the generated client. > @jason810496 Please see below Claude's analysis of the point you raised. I am going to add the extra test. Please let me know if you think this is insufficient. Yes, the analysis makes sense to me, then I think we don't need `field_serializer` at all, as the type is still string before we adding the `field_serializer`. And, yes, having the test to ensure the compatibility would be nice. Thanks! -- 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]
