kaxil commented on PR #72981: URL: https://github.com/apache/airflow/pull/72981#issuecomment-5940868309
Closing this. On #68382, ashb said that if `port` is optional then `0` shouldn't be accepted either, and asked how existing rows with now-invalid ports should be handled; this accepts `port=0` (`test_port_accepts_boundary_values`) and doesn't address existing data. It also covers only the ORM model, while #68700 is open for the same issue across all the layers the issue lists and is already under review. -- 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]
