Skip to content

Improvements to clarify type using TypedParameterValue#2461

Closed
heowc wants to merge 1 commit into
spring-projects:mainfrom
heowc:gh-2370
Closed

Improvements to clarify type using TypedParameterValue#2461
heowc wants to merge 1 commit into
spring-projects:mainfrom
heowc:gh-2370

Conversation

@heowc

@heowc heowc commented Mar 8, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • For Hibernate provider, create HibernateJpaParametersParameterAccessor not JpaParametersParameterAccessor
  • Create TypedParameterValue by fetching parameter type from HibernateJpaParametersParameterAccessor

Close #2370

  • You have read the Spring Data contribution guidelines.
  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.
  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

@heowc
Improvements to clarify type using TypedParameterValue
c42c2bd 
- For Hibernate provider, create `HibernateJpaParametersParameterAccessor` not `JpaParametersParameterAccessor`
- Create `TypedParameterValue` by fetching parameter type from `HibernateJpaParametersParameterAccessor`
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 8, 2022
schauder pushed a commit that referenced this pull request Mar 10, 2022
@heowc @schauder
Null arguments now get wrapped in a TypedParameterValue for Hibernate.
3c804cf 
This avoids problems where a JDBC driver fails to determine the correct type for such values.

Closes #2370
Original pull request #2461
schauder added a commit that referenced this pull request Mar 10, 2022
@schauder
Polishing.
dfb9d7a 
Replaced type check by method providing the correct JpaParameterParameterAccessor.
Formatting.

See #2370
Original pull request #2461
schauder pushed a commit that referenced this pull request Mar 10, 2022
@heowc @schauder
Null arguments now get wrapped in a TypedParameterValue for Hibernate.
60ae66d 
This avoids problems where a JDBC driver fails to determine the correct type for such values.

Closes #2370
Original pull request #2461
schauder added a commit that referenced this pull request Mar 10, 2022
@schauder
Polishing.
065d77a 
Replaced type check by method providing the correct JpaParameterParameterAccessor.
Formatting.

See #2370
Original pull request #2461
@schauder

schauder commented Mar 10, 2022

Copy link
Copy Markdown
Contributor

Thanks for your work, that's merged.

@schauder schauder closed this Mar 10, 2022
@heowc heowc deleted the gh-2370 branch March 14, 2022 09:07
@cigaly

cigaly commented May 15, 2022

Copy link
Copy Markdown
Contributor

There is a problem with this implementation. When EntityManager parameter in constructor is proxy created by SharedEntityManagerCreator, em.unwrap(Session.class) may (or will?) throw IllegalStateException with message "No transactional EntityManager available".

Attached is small change that can fix that problem.

HibernateJpaParametersParameterAccessor.txt

@heowc heowc mentioned this pull request May 19, 2022
Closed
This was referenced May 30, 2022
Closed
Closed
@quaff quaff mentioned this pull request Jun 30, 2022
Closed
gregturn added a commit that referenced this pull request Jul 13, 2022
@gregturn
Properly handle null values inside queries using LIKE or CONTAINS.
3d8b287 
Null values are wrapped with a special handler when interacting with Hibernate. However, this becomes an issue for queries when LIKE or CONTAINS are applied. In this situation, the null needs to be condensed into an empty string and any wildcards can then be applied with expected results.

Closes #2548, #2570.
Supercedes: #2585.
Related: #2461, #2544#
@gregturn

gregturn commented Jul 13, 2022

Copy link
Copy Markdown
Contributor

@cigaly Please check out the snapshots to see if your issue still persists. If so, please open a new ticket.

@davin111 davin111 mentioned this pull request Aug 15, 2022
Merged
gregturn added a commit that referenced this pull request Sep 23, 2022
@gregturn
Properly handle null values inside queries using LIKE or CONTAINS.
f0216b0 
Null values are wrapped with a special handler when interacting with Hibernate. However, this becomes an issue for queries when LIKE or CONTAINS are applied. In this situation, the null needs to be condensed into an empty string and any wildcards can then be applied with expected results.

Closes #2548, #2570.
Supercedes: #2585.
Related: #2461, #2544#
gregturn added a commit that referenced this pull request Nov 7, 2022
@gregturn
Properly handle null mixed with LIKE.
cdb32ba 
Resolves: #2653.
Related: #2548, #2683, #2655, #2461, possibly #2544
gregturn added a commit that referenced this pull request Nov 9, 2022
@gregturn
Properly handle null mixed with LIKE.
34e16e3 
Resolves: #2653.
Related: #2548, #2683, #2655, #2461, possibly #2544
gregturn added a commit that referenced this pull request Nov 9, 2022
@gregturn
Properly handle null mixed with LIKE.
e8c4c53 
Resolves: #2653.
Related: #2548, #2683, #2655, #2461, possibly #2544
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Improvements to clarify type using TypedParameterValue

5 participants

@heowc @schauder @cigaly @gregturn @spring-projects-issues