uranusjr opened a new pull request, #27251: URL: https://github.com/apache/airflow/pull/27251
See discussion in #27209 for context. The class is intended to work as a "lazy list" to avoid pulling a ton of XComs unnecessarily, but if it's pushed into XCom, the user should be aware of the performance implications, and this avoids leaking the implementation detail. An alternative "solution" to the issue would be to accept this implementation detail should be exposed to the user (it already is), and simply document the caveat. This approach is explored in #27250. -- 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]
