potiuk commented on PR #27250: URL: https://github.com/apache/airflow/pull/27250#issuecomment-1292087214
This looks good - but if the #27251 is solid, I have strong preference for that one. Users don't read all the details in the documentation and they will expect it to "just work" . They don't even realise (and they should not) there is a lazy-proxy used. The whole taskflow API is there precisely to hide the implementation details (Xcom.push, Xcom.pull and the like) from the user, so this is entirely expected they have no idea about this detail. And since not many users read documentation, they will open an issue instead most likely (which we want to at least try to avoid). Another option (if we cannot make #27251 solid) will be to make a very explict error message when they try to push LazyXCom and point them to this documentation directly - then they will have a chance to read it and even if they copy&paste the error message in the issue, we will just close the issue and tell them to follow the advice they got. -- 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]
