Skip to content
New issue

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

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

Already on GitHub? # to your account

Annotation for sql type [DATAJDBC-239] #464

Closed
spring-projects-issues opened this issue Jul 20, 2018 · 5 comments
Closed

Annotation for sql type [DATAJDBC-239] #464

spring-projects-issues opened this issue Jul 20, 2018 · 5 comments
Assignees
Labels
status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement

Comments

@spring-projects-issues
Copy link

Jens Schauder opened DATAJDBC-239 and commented

We need a way to specify the java.sql.Types value to be used for binding parameters.

This applies to

  • columns/properties
  • converters
  • method parameters

The idea is to have a single annotation that can be used in all these places


Issue Links:

3 votes, 5 watchers

@spring-projects-issues
Copy link
Author

Jens Schauder commented

When this is implemented for converters it should solve DATAJDBC-148

@spring-projects-issues
Copy link
Author

Jens Schauder commented

The converters part is currently getting implemented by DATAJDBC-327

@spring-projects-issues
Copy link
Author

Johannes Edmeier commented

Without this feature it's also currently not possible to use custom value types as Ids for aggregates with nested entities.
Reason: org.springframework.data.jdbc.core.convert.DefaultDataAccessStrategy.findAllByPath() uses the id as query parameters, which currently don't get converted

@spring-projects-issues
Copy link
Author

Jens Schauder commented

Johannes Edmeier is this still the case for the master branch (1.1 milestone) if so please create a separate issue for this

@spring-projects-issues
Copy link
Author

Jens Schauder commented

It currently doesn't look like this is needed

@spring-projects-issues spring-projects-issues added status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement labels Dec 31, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

2 participants