FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GH-964: Fix IndexOutOfBoundsException in Array.getResultSet() for JDBC clients by xborder · Pull Request #965 · apache/arrow-java · GitHub

GH-964: Fix IndexOutOfBoundsException in Array.getResultSet() for JDBC clients - #965

Merged
jbonofre merged 2 commits into
apache:mainfrom
xborder:fix-array-resultset
Jan 17, 2026
Merged

GH-964: Fix IndexOutOfBoundsException in Array.getResultSet() for JDBC clients#965
jbonofre merged 2 commits into
apache:mainfrom
xborder:fix-array-resultset

Conversation

xborder commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

What's Changed

  • Fixed JDBC specification in ArrowFlightJdbcArray.getResultSet() that caused IndexOutOfBoundsException in JDBC clients like DBeaver when reading array columns
  • The method returned a single-column ResultSet containing only array values, but JDBC spec requires a 2-column format
  • Not it returns two columns:
    • Column 1 (INDEX): 1-based element indices per JDBC specification
    • Column 2: The actual array element values

Closes #964.

This comment has been minimized.

xborder commented Jan 12, 2026

Copy link
Copy Markdown
Contributor Author

Could this be considered a breaking-change even though it is a fix to comply with the JDBC spec? How should this be handled?

Copy link
Copy Markdown
Member

I think it's not a breaking change to make it comply with the spec.

Can we make the title descriptive of the changes?

xborder changed the title GH-964: Return ResultSet GH-964: Fix IndexOutOfBoundsException in Array.getResultSet() for JDBC clients Jan 12, 2026

jbonofre left a comment
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

If it could be seen as a breaking change from a code perspective, I don't think it's a breaking change as it's aligned with the spec (I would rather consider as a bug fix).

jbonofre added the bug-fix PRs that fix a big. label Jan 16, 2026
github-actions Bot added this to the 19.0.0 milestone Jan 16, 2026
jbonofre merged commit 349d402 into apache:main Jan 17, 2026
57 of 63 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix PRs that fix a big.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DBeaver fails to display columns of array type

3 participants


Back | FazBrowse Home | New Git URL