dstandish commented on code in PR #28191:
URL: https://github.com/apache/airflow/pull/28191#discussion_r1049218067


##########
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
+        # query object, but there's no good way to extract query information.
+        # The SQL string for DBAPI is the best we can get. THeoratically we can

Review Comment:
   ```suggestion
           # The SQL string for DBAPI is the best we can get. Theoratically we 
can
   ```



-- 
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

Reply via email to