Description Sascha Baumeister 2015-02-04 06:46:37 EST

@CacheIndex can be annotated on entities and their fields/accessors, therefore I agree that @CacheIndex.updateable is required in general. However, it currently defaults to true in any case, causing the need to specify fully redundant information if annotated on a field/accessor whenever an @Column annotation is present as well.

Requiring the configuration or annotation of redundant information is bad practice because it is prone to contradictions, like @CacheIndex.updateable=true but @Column.updatable=false. Therefore, I believe the default should be changed in case @CacheIndex is annotated on a field/accessor, and an @Column annotation is present as well on said field/accessor: under these circumstances, @CacheIndex.updatable should default to @Column.updatable, which itself defaults to true.