Auto configuration for Spring Data Envers

[scordio]

When using Spring Data Envers with Spring Boot, the EnversRevisionRepositoryFactoryBean has to be configured via @EnableJpaRepositories:

@EnableJpaRepositories(repositoryFactoryBeanClass = EnversRevisionRepositoryFactoryBean.class)

Maybe an auto configuration could take care of it. Would it make sense?

In case it does, I would be happy to prepare a pull request.

[philwebb]

@mp911de Hi Mark, what are your thoughts on this? Is it a sensible thing to auto-configure?

[mp911de]

Adding also @schauder to the conversation.

Envers repositories assume that the entity is audited and allows revision retrieval. From what I’ve seen we cannot expect that this assumption holds true for each application. Instead, only some entities are audited.

Since I’m not exactly sure on the impact on non-audited entities, I’d like Jens to chime in.

[scordio]

Each repository would still need to extend from org.springframework.data.repository.history.RevisionRepository in order to get revision retrieval methods, so the application still has to properly declare a repository for each audited entity.

If I am not mistaken, a repository extending RevisionRepository cannot work with Envers if the above configuration is not applied. At the same time, the configuration above should have no impact on repositories not extending from RevisionRepository.

My idea is that when EnversRevisionRepositoryFactoryBean (or any other class coming from spring-data-envers) is in the classpath , most likely the application wants to use Spring Data JPA together with Envers, hence my proposal about the auto configuration.

[schauder]

Pro auto configuration:

• The impact on non audited entities should be minimal.
  Especially it is well separated from the JPA infrastructure.
  This impression is supported by the fact that I can't remember a single issue where Spring Data Envers triggered a bug in Spring Data JPA.

Contra:

• Spring Data Envers is rather low usage (~1.5% downloads compared to Spring Data JPA)
• I expect it to be common, that only some entities are audited. As mentioned above this SHOULD not be an issue.

I'm not sure what the criteria are for inclusion of auto configuration, but my gut reaction is: "Yeah, why not"