Skip to content
This repository has been archived by the owner on May 25, 2020. It is now read-only.

Treated path operation support and testing #4

Closed
jwgmeligmeyling opened this issue Feb 5, 2020 · 1 comment
Closed

Treated path operation support and testing #4

jwgmeligmeyling opened this issue Feb 5, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@jwgmeligmeyling
Copy link
Owner

No description provided.

@jwgmeligmeyling jwgmeligmeyling added the enhancement New feature or request label Feb 5, 2020
@jwgmeligmeyling jwgmeligmeyling self-assigned this Feb 5, 2020
@jwgmeligmeyling
Copy link
Owner Author

jwgmeligmeyling commented Feb 15, 2020

Turns out that we didn't need any special handling for treat operations, as these are embedded in the serialised join expression and accepted by the Blaze-Persistence JPQL Next parser.

I did find out though that QueryDSL only supported TREAT operations in the FROM clause. JPA also describes the possibility to use TREAT operations in the WHERE clause using treated paths. Some JPA providers even support TREAT in SELECT, ORDER BY, etc. Blaze-Persistence may rewrite TREAT to JOINS in unsupported cases (Blazebit/blaze-persistence#409).

It turned out to be rather difficult to implement TREAT support in the builder API, because the implementation of Path and PathImpl is rather closed for extension. It however was relatively easy to fix in QueryDSL itself. For this pull request querydsl/querydsl#2530 has been created.

@jwgmeligmeyling jwgmeligmeyling moved this from To do to Done in Initial release Feb 15, 2020
jwgmeligmeyling added a commit that referenced this issue Feb 15, 2020
@jwgmeligmeyling jwgmeligmeyling added this to the Initial release milestone Mar 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

1 participant