Skip to content

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