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.

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

5.3.8. Modifying Queries: unclear reference to "custom methods" [DATAJPA-1793] #2088

spring-projects-issues opened this issue Oct 9, 2020 · 3 comments


Copy link

@spring-projects-issues spring-projects-issues commented Oct 9, 2020

Chealer opened DATAJPA-1793 and commented

Section 5.3.8. Modifying Queries ends with:

The @Modifying annotation is only relevant in combination with the @Query annotation. Derived query methods or custom methods do not require this Annotation.

I do not understand what this considers as a "custom method".

I suggest to either define what custom methods are or to use different terminology

Affects: 2.3.4 (Neumann SR4)

Reference URL:

Backported to: 2.3.5 (Neumann SR5), 2.2.11 (Moore SR11)

Copy link

@spring-projects-issues spring-projects-issues commented Oct 12, 2020

Jens Schauder commented

This is referring to

Adding a link to that section is probably a good idea

Copy link

@spring-projects-issues spring-projects-issues commented Nov 17, 2020

Chealer commented

Thank you Jens, I see.

How was this resolved? I see that the quoted sentences are unchanged. However, I notice there is already a link to the section mentioned by Jens not long before (at the start of the section with the unclear reference). It would still clarify to either rephrase the sentence, add another link to the relevant section, define custom methods somewhere, or mention custom methods by name in the section about them

Copy link

@spring-projects-issues spring-projects-issues commented Nov 23, 2020

Jens Schauder commented

I resolved it with introducing the term "custom method" explicitly. a99a087

This way searching for the term if it isn't immediately clear will yields useful results.
If you'd like to see further changes please open a new ticket and a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.