o-nikolas commented on PR #71619:
URL: https://github.com/apache/airflow/pull/71619#issuecomment-5321323515

   > Thanks so much for catching those handler and multi-statement issues, 
@o-nikolas! Really appreciate you (and your AI agent 😄) taking the time to dig 
into pyexasol's ExaStatement internals — that's a subtle one I definitely 
wouldn't have caught on my own. Just pushed a fix for everything:
   > 
   >     * Switched to `exasol_fetch_all_handler` on all three 
SQLExecuteQueryOperator tasks
   > 
   >     * Split the CREATE TABLE + INSERT into separate tasks so we're not 
relying on multi-statement execution
   > 
   >     * Removed the now-stale skip entry in test_project_structure.py (since 
we finally have an example!)
   > 
   >     * Added the missing how-to-guide link in provider.yaml, and synced 
get_provider_info.py to match
   >       This is my first Airflow contribution, so thanks for the patience 
walking through it! Whenever someone with write access gets a chance, the 
workflow run is waiting on approval to actually execute. No rush at all
   
   Thanks for the fixes! Have you run this against a live Exasol+AWS 
environment for testing?


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

Reply via email to