QueryDslPredicateExecutor delete [DATACMNS-553]

Lovro Pandžić opened DATACMNS-553 and commented

Add a delete method to QueryDslPredicateExecutor that accepts Predicate as an argument

---

Issue Links:

• DATAJPA-954 Delete by Predicate (“is depended on by”)

[mp911de]

@lpandzic we discussed that topic with our team. Right now, QuerydslPredicateExecutor contributes read-only methods (findBy…) to a repository. Adding a delete method would suddenly add a modifying flavor to all repositories that implement the interface thus we aren’t convinced about the ratio between useful and introducing unwanted functionality.

We will revisit that topic during our efforts for Spring Data 3.0 where we want to investigate further on a read-only/read-write method split for repositories.

[mp911de]

Part of the challenge here is that we cannot provide generic update objects. We cannot accept an entity for an update as it’s not obvious what that should express. Also, we do not have generic update objects that would e.g. set individual fields, add values to an array, increment values as these operations aren’t supported consistently across all store modules. So the only thing that would be left over is a delete method and there the ratio between usefulness and effort isn’t in the right balance.