ashb commented on code in PR #47339:
URL: https://github.com/apache/airflow/pull/47339#discussion_r1987138614
##########
airflow/models/xcom.py:
##########
@@ -161,27 +183,6 @@ def set(
if dag_run_id is None:
raise ValueError(f"DAG run not found on DAG {dag_id!r} with ID
{run_id!r}")
- # Seamlessly resolve LazySelectSequence to a list. This intends 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.
- if isinstance(value, LazySelectSequence):
Review Comment:
We can probably delete that class now too, as I think this was the last
usage of it?
--
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]
