guan404ming commented on PR #49339: URL: https://github.com/apache/airflow/pull/49339#issuecomment-2816669851
Hi @eladkal after my investigation, I found the the cursor execution logic is different for `pandas` and `polars`. Reference to the `polars.read_database` source code, there is a abstraction for executer when calling `read_database`: https://github.com/pola-rs/polars/blob/5c114f0f32a62bf796629e4ccf01b93146ae22c4/py-polars/polars/io/database/functions.py#L247-L256 Thus we have also need to make the abstraction, while pandas just do `cursor.fetchall()` directly which meaning that the current implement for test is suitable for pandas. Also, we need to skip the polars test for `low_deps` and `AF2` in ci, so I split them into two functions to handle the test here -- 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]
