steffenvan commented on code in PR #1327:
URL: https://github.com/apache/jackrabbit-oak/pull/1327#discussion_r1503743424


##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/index/ElasticIndexHelper.java:
##########
@@ -52,8 +53,12 @@ class ElasticIndexHelper {
      * Changes not breaking compatibility should increment the minor version 
(old queries still work, but they might not
      * use the new feature).
      * Changes that do not affect queries should increment the patch version 
(eg: bug fixes).
+     * <p>
+     * WARN: Since this information might be needed from external tools that 
don't have a direct dependency on this module, the
+     * actual version needs to be set in oak-search.
      */
-    protected static final String MAPPING_VERSION = "1.1.0";
+    protected static final String MAPPING_VERSION = 
FulltextIndexConstants.INDEX_VERSION_BY_TYPE.

Review Comment:
   Hmm I'm wondering why the default one is `1.0.0` instead of `1.1.0`?



-- 
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: dev-unsubscr...@jackrabbit.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to