ephraimbuddy commented on PR #25532: URL: https://github.com/apache/airflow/pull/25532#issuecomment-1206112702
> I looked at the code and this seems to be the only place (except tests) where we use `session.bulk_*` operations so we should be pretty save. > > However I think it would be good if we add a description of the problem here and link to the superset issue in a comment here just to capture it somewhere in the code (I am thinking about my fufure self in a year, hitting similar issue and trying to find out what was it about when I recall that something similar happened in the past) :) Look how I did it now. I feel it's much better and understandable -- 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]
