dabla commented on PR #38715: URL: https://github.com/apache/airflow/pull/38715#issuecomment-2042181690
> So it’s due to not all DBAPI libs support the flag? That makes sense. I wonder if there’s a smarter way to detect this. The current implementation should be fine though since it is designed as an opt-in. > > The `fast_executemany` flag should be an explicit argument in `__init__`, not popped out of `kwargs`. Yes indeed, a from what I've found even if you call it but is not supported, it shouldn't raise an exception, but exception could then occur at runtime hence why I've put it as a checked condition. Ok will put it as an expliciet parameter -- 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]
