Fixed
Details
Assignee
Yoann RodièreYoann RodièreReporter
Yoann RodièreYoann RodièreSprint
NoneFix versions
Priority
Major
Details
Details
Assignee
Yoann Rodière
Yoann RodièreReporter
Yoann Rodière
Yoann RodièreSprint
None
Fix versions
Priority
Created March 26, 2019 at 11:46 AM
Updated April 4, 2019 at 9:41 AM
Resolved March 27, 2019 at 1:51 PM
As far as I can tell, we never take into account the fact that converters are ignored when validating the compatibility of fields when targeting multiple indexes.
Our tests pass, but just because I made a poor job of designing the converters: they happen to always be compatible...
We should:
Change
org.hibernate.search.integrationtest.backend.tck.testsupport.util.ValueWrapper#toIndexFieldConverter
andorg.hibernate.search.integrationtest.backend.tck.testsupport.util.ValueWrapper#fromIndexFieldConverter
to pass some parameter that will ensure converters are only deemed compatible when we need them to beChange
org.hibernate.search.backend.elasticsearch.search.projection.impl.ElasticsearchSearchProjectionBuilderFactory#field
and similar so that they do not fail if we pass ProjectionConverter.DISABLED and the projection converters are not compatible.