Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: EXPOSED-719 H2 upsert operation converts arrays to string #2395

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

obabichevjb
Copy link
Collaborator

Description

  • What: Upsert command fails for H2 if it contains array column.
  • Why: The problem is that H2 has problems with type inference and fails with Data conversion error converting for such columns
  • How: The only way I found is cast explicitly parameter markers for array values on H2 database. The list of particular types was added, and the test checks that everything works for these types, but in general such a cast could not work for other types (for example it was not working for BigDecimal)

Type of Change

Please mark the relevant options with an "X":

  • Bug fix

Affected databases:

  • H2

Checklist

  • Unit tests are in place
  • The build is green (including the Detekt check)
  • All public methods affected by my PR has up to date API docs
  • Documentation for my change is up to date

Related Issues

EXPOSED-719 H2 upsert operation converts arrays to string

@obabichevjb obabichevjb self-assigned this Feb 5, 2025
@obabichevjb obabichevjb requested a review from joc-a February 5, 2025 12:15
@obabichevjb obabichevjb force-pushed the obabichev/exposed-719-h2-array-upsert branch from eb32ccc to ba4b37f Compare February 5, 2025 12:15
@obabichevjb obabichevjb merged commit c3029fe into main Feb 6, 2025
5 checks passed
@obabichevjb obabichevjb deleted the obabichev/exposed-719-h2-array-upsert branch February 6, 2025 12:49
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