uranusjr commented on pull request #16931: URL: https://github.com/apache/airflow/pull/16931#issuecomment-950539145
Generally looks good to me now! One thing I'm wondering is how `None` and `""` differ semantically. From the current implementation in HTML, they seem to be equal? How could a difference be introduced in the future? If there are no differences in semantics for our current use cases, IMO it'd be a good idea to *forbid* None (NULL in database) for now to save it for potential use cases that may come up. -- 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]
