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.

Sign up for GitHub

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

Jump to bottom

Support to decode primitive integers (and other primitives?) #85

Open
odrotbohm opened this issue Aug 13, 2019 · 2 comments
Open

Support to decode primitive integers (and other primitives?) #85

odrotbohm opened this issue Aug 13, 2019 · 2 comments
Labels
type: spec A change mandated by the specification

Comments

@odrotbohm
Copy link

I just ran into an issue trying to call row.get(…, int.class) with the error message saying int is not a supported type. Switching to Integer.class got it working but I wonder whether we should be able to decode primitives OOTB to avoid boxing.
@mp911de
Copy link
Member

mp911de commented Aug 13, 2019

R2DBC API can return currently only boxed (wrapper) types for primitives. We should have a definition of what happens with fields that are null when asking for a primitive. This ticket touches the spec so I wonder whether we should elevate it to the SPI project to get consistent behavior across driver implementations.

@gregturn gregturn mentioned this issue Aug 14, 2019
Closed
@gregturn
Copy link
Contributor

May be related to #77 .

@mp911de mp911de added the type: spec A change mandated by the specification label Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: spec A change mandated by the specification
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@odrotbohm @gregturn @mp911de