Joffreybvn commented on code in PR #32319:
URL: https://github.com/apache/airflow/pull/32319#discussion_r1252224880
##########
airflow/providers/common/sql/hooks/sql.py:
##########
@@ -372,6 +372,15 @@ def run(
else:
return results
+ def _make_serializable(self, result: Any) -> Any:
+ """Ensure the data returned from a SQL command is JSON-serializable.
+
+ This method is intended to be overridden by subclasses of the
+ `DbApiHook`. Its purpose is to transform the result of a SQL command
+ into a JSON-serializable format.
Review Comment:
In most cases, this won't be overidden, and indeed the original input is
returned as output. For what I know, the only cases where they are custom Row
objects are in Databricks and PyODBC. I'll edit the docstring.
--
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]
