dstandish commented on code in PR #28191:
URL: https://github.com/apache/airflow/pull/28191#discussion_r1049217806
##########
airflow/models/xcom.py:
##########
@@ -723,6 +723,20 @@ def __eq__(self, other: Any) -> bool:
return all(x == y for x, y in z)
return NotImplemented
+ def __getstate__(self) -> Any:
+ # We don't want to get into the trouble to pickle the entire SQLAlchemy
Review Comment:
```suggestion
# We don't want to get into the trouble to pickle the entire
SQLAlchemy
```
it's a little unclear whether you mean it's a big effort to do, or it's a
problematic thing to do.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
