Skip to content

Add FluentQuery support to QuerydslPredicateExecutor and QueryByExampleExecutor. #2360

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.

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

Closed
wants to merge 4 commits into from

Conversation

michael-simons
Copy link
Collaborator

Pinging @mp911de. I don't expect an internal review from you, but I'd like to assign this to you for when you have merged the changes in common needed to accommodate this.

…`QueryByExampleExecutor`.

This change uses the existing `FluentOperations` to cater for the above usescases. It introduces a new `matching` operation for taking in a `QueryFragmentsAndParameters` on those operations.

That operation is of limited external use, but necessary to implement that feature without adding more overhead than necessary.

This closes #2343.
@mp911de mp911de added the type: enhancement A general enhancement label Oct 8, 2021
mp911de pushed a commit that referenced this pull request Oct 8, 2021
…`QueryByExampleExecutor`.

This change uses the existing `FluentOperations` to cater for the above usescases. It introduces a new `matching` operation for taking in a `QueryFragmentsAndParameters` on those operations.

That operation is of limited external use, but necessary to implement that feature without adding more overhead than necessary.

This closes #2343.
Original pull request: #2360.
mp911de pushed a commit that referenced this pull request Oct 8, 2021
mp911de added a commit that referenced this pull request Oct 8, 2021
@mp911de
Copy link
Member

mp911de commented Oct 8, 2021

That's merged now.

@mp911de mp911de closed this Oct 8, 2021
@mp911de mp911de deleted the issue/2343 branch October 8, 2021 12:06
@michael-simons
Copy link
Collaborator Author

Woohoo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add FluentQuery support to QuerydslPredicateExecutor and QueryByExampleExecutor
2 participants