Skip to content

Do not convert binary data in List<byte[]> to array SQL type #1903

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

Closed
wants to merge 2 commits into from

Conversation

schauder
Copy link
Contributor

@schauder schauder commented Oct 2, 2024

byte[] is mapped to BINARY data and is not considered tuple data.

The explicit check for byte[] is not very nice but matches the special handling in MappingJdbcConverter.writeJdbcValue

Closes #1900

byte[] is mapped to BINARY data and is not considered tuple data.

The explicit check for byte[] is not very nice but matches the special handling in MappingJdbcConverter.writeJdbcValue

Closes #1900
@mp911de
Copy link
Member

mp911de commented Oct 2, 2024

With an outlook to overhaul collection processing, that seems an okay workaround.

@mp911de mp911de added this to the 3.3.5 (2024.0.5) milestone Oct 2, 2024
@mp911de mp911de added the type: regression A regression from a previous release label Oct 2, 2024
@mp911de mp911de changed the title Do not convert binary data in List<byte[]>. Do not convert binary data in List<byte[]> to array SQL type Oct 2, 2024
mp911de pushed a commit that referenced this pull request Oct 2, 2024
byte[] is mapped to BINARY data and is not considered tuple data.

The explicit check for byte[] is not very nice but matches the special handling in MappingJdbcConverter.writeJdbcValue

Closes #1900
Original pull request: #1903
mp911de added a commit that referenced this pull request Oct 2, 2024
Refine tests. Extend comment for future-me.

See #1900
Original pull request: #1903
mp911de pushed a commit that referenced this pull request Oct 2, 2024
byte[] is mapped to BINARY data and is not considered tuple data.

The explicit check for byte[] is not very nice but matches the special handling in MappingJdbcConverter.writeJdbcValue

Closes #1900
Original pull request: #1903
mp911de added a commit that referenced this pull request Oct 2, 2024
Refine tests. Extend comment for future-me.

See #1900
Original pull request: #1903
@mp911de mp911de closed this Oct 2, 2024
@mp911de mp911de deleted the issue/1900-byte-array-to-uuid branch October 2, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: regression A regression from a previous release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Query method with bind parameter List<byte[]> does no longer work
2 participants