Add delete by id method to repository interface [DATACMNS-35] about spring-data-commons HOT 2 CLOSED

Oliver Drotbohm commented

There's two major reasons we don't expose an delete(ID id) yet. The first one is that the EntityManager only exposes a remove(Object entity) int the first place so that it is rather simple to do that. The second point aligns to you're requirement of not loading the entity at all. To achieve that we'd have to trigger a JQPL query which - according to the spec - does not cause related entities to be updated or deleted. This would the create a delta in the semantics of delete(T entity) and delete(ID id) which is a bit inconvenient.

Having that said, I perfectly get your point of deleting an entity by id just from a convenience point of view. So I think we could add an delete(ID id) method that uses findOne(ID id) under the hood and would thus save you one call to the repository plus execute the operation in a single transaction then

from spring-data-commons.

Oliver Drotbohm commented

Added method to the interface as well as implementations for JPA and Mongo

from spring-data-commons.