Improve javadoc on QueryDslPredicateExecutor [DATACMNS-577]

[spring-projects-issues]

Lovro Pandžić opened DATACMNS-577 and commented

Methods in the QueryDslPredicateExecutor should be better documented.
For example:

• findOne - what happens if it finds more than one result and what happens if it does not find any - does it return a null or does it throw an exception?
• findAll - how can it fail, can it return a null?

Fix params that are missing description.

Otherwise this causes possibly unneeded null checks and exception checks in the client code because clients can't know what to expect. This might also be applied to other repository interface definitions

---

Affects: 1.9 GA (Evans)

Referenced from: pull request #96

Backported to: 1.9.1 (Evans SR1), 1.8.5 (Dijkstra SR5)

[spring-projects-issues]

Thomas Darimont commented

Please revise

[spring-projects-issues]

Oliver Drotbohm commented

See the minor comments on the PR