pateash commented on pull request #16931: URL: https://github.com/apache/airflow/pull/16931#issuecomment-951058516
> 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. That's a really good point from future perspective, We should really use empty string. Doing the changes. -- 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]
