potiuk commented on PR #27251: URL: https://github.com/apache/airflow/pull/27251#issuecomment-1296893148
I personally think this is far better approach than https://github.com/apache/airflow/pull/27250 - most users don't read the documentation and some of them will raise an issue if they find the problem. Implementing automated coertion is best solution (the best way to solve the problem is to avoid it altogether). Yes, it has some obvious performance consequences, but then one thing that we could do is we could write a warning in the log that this automated coertion has potential performance problems (we can even be smart and only print it if coertion takes more than arbitrary number of seconds to not introduce too much noice. And of course documenting thos performance issues in the docs (then the log could have "See more URL_TO_THE_DOCS" pointing to the right part of documentation. For me this is the best "user-friendly" and "maintainer-friendly" approach we can come up with. -- 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]
