Skip to content

DRILL-8551: Fix Informix JDBC queries with catalog-qualified table names - #3065

Merged
cgivre merged 1 commit into
apache:masterfrom
014-code:issue-3019-informix-jdbc
Jul 28, 2026
Merged

DRILL-8551: Fix Informix JDBC queries with catalog-qualified table names#3065
cgivre merged 1 commit into
apache:masterfrom
014-code:issue-3019-informix-jdbc

Conversation

@014-code

Copy link
Copy Markdown
Contributor

Summary

  • Add an Informix-specific JDBC dialect path for SQL generation.
  • Drop the JDBC catalog from remote Informix table identifiers while preserving schema-qualified names.
  • Add regression coverage for Informix table identifier generation and dialect selection.

Tests

  • mvn -s C:\Users\14\AppData\Local\Temp\maven-no-mirror-settings.xml -gs C:\Users\14\AppData\Local\Temp\maven-no-mirror-settings.xml -P '!brotli-amd64,!brotli-x86_64' -Drat.skip=true -Dcheckstyle.skip=true -Dtest=org.apache.drill.exec.store.jdbc.informix.TestInformixJdbcDialect -DfailIfNoTests=false -pl contrib/storage-jdbc test
  • Result: Tests run: 4, Failures: 0, Errors: 0, Skipped: 0

Fixes #3019

@cgivre cgivre added bug minor-update backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there labels Jul 26, 2026
@cgivre cgivre changed the title Fix Informix JDBC queries with catalog-qualified table names DRILL-8551: Fix Informix JDBC queries with catalog-qualified table names Jul 26, 2026
@014-code
014-code force-pushed the issue-3019-informix-jdbc branch from 5bec6f1 to 9cb0a9b Compare July 26, 2026 03:10

@cgivre cgivre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1. Thanks for the contribution!

@cgivre
cgivre merged commit 2a4a79b into apache:master Jul 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there bug minor-update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error querying Informix database

2 participants