FrancoisZhang commented on a change in pull request #494:
URL: https://github.com/apache/jackrabbit-oak/pull/494#discussion_r808218965
##########
File path:
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/index/ElasticIndexHelper.java
##########
@@ -276,6 +276,11 @@ private static void mapIndexRules(ElasticIndexDefinition
indexDefinition, XConte
mappingBuilder.endObject();
}
+ mappingBuilder.startObject(ElasticIndexDefinition.DYNAMIC_BOOST_TAGS)
+ .field("type", "text")
+ .field("analyzer", "oak_analyzer")
+ .endObject();
+
Review comment:
@fabriziofortino thanks, that makes sense. I will change that way.
I actually think the origin way I put here with fixed field should also
works, the reason the query is cross fields and don't really care if they're
stored in single field or multiple fields. I could also put multiple fields,
both should works IMO and fixed will be easier.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
