priy-anshugupta commented on issue #40066: URL: https://github.com/apache/airflow/issues/40066#issuecomment-6684003690
I'd like to help with this if it is still available. Is the original reporter still planning to implement it? The dialect-specific bind parameters in the example seem to be the main constraint: converting a statement with `str()` would not be enough. Before writing a patch, I'd like to confirm whether maintainers prefer compiling statements for the hook's SQLAlchemy dialect while retaining DB-API execution, or a separate SQLAlchemy execution path. That choice also needs to account for templating, transaction handling and result handlers. If this is still wanted, I would start with a bound-parameter regression case for SQLite and PostgreSQL, keeping the existing string-SQL behavior unchanged. I have read the discussion and operator code, but have not implemented or tested a patch yet. --- Drafted-by: OpenAI Codex (GPT-6) (no human review before posting) -- 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]
