Skip to content

feat: add configurable DML row counts and skip disabled test_get_objects_ instead of strict xfails - #271

Merged
Mandukhai-Alimaa merged 4 commits into
adbc-drivers:mainfrom
Mandukhai-Alimaa:feat/dml-statement-rows-affected-flag
Jul 28, 2026
Merged

feat: add configurable DML row counts and skip disabled test_get_objects_ instead of strict xfails#271
Mandukhai-Alimaa merged 4 commits into
adbc-drivers:mainfrom
Mandukhai-Alimaa:feat/dml-statement-rows-affected-flag

Conversation

@Mandukhai-Alimaa

Copy link
Copy Markdown
Contributor

Some backends, such as Databend, report 0 rows affected for DML statements. Add a driver feature flag so the validation suite can expect this behavior.

Skipped test_get_objects_ instead of marking strict xfail when disabled. In the MySQL driver, unfiltered GetObjects uses a plain query and may succeed on Databend, while filtered GetObjects uses bound parameters and falls back to prepared statements, which Databend does not support. With strict xfail, unfiltered cases can pass and turn into misleading XPASS failures even though get_objects is disabled for that backend. Skipping is clearer.

Comment thread adbc_drivers_validation/model.py Outdated
@Mandukhai-Alimaa
Mandukhai-Alimaa merged commit f0c3574 into adbc-drivers:main Jul 28, 2026
3 checks passed
Mandukhai-Alimaa added a commit to adbc-drivers/mysql that referenced this pull request Jul 29, 2026
…with Databend (#122)

Only inject MySQL's session `time_zone` parameter for MySQL and MariaDB.
Add Databend validation coverage and workflow matrix entries.

Running the validation suite before the fix reproduces the issue, and
after the fix, it runs successfully and connects to Databend.

Closes #61 

This needs to be merged in first
adbc-drivers/validation#271
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants