cpoerschke commented on code in PR #3053:
URL: https://github.com/apache/solr/pull/3053#discussion_r2288462023


##########
solr/core/src/test-files/solr/collection1/conf/schema.xml:
##########
@@ -471,12 +471,13 @@
     </analyzer>
   </fieldType>
   <!-- 
-    Example of using PathHierarchyTokenizerFactory at query time, so
-    queries for paths match documents at that path, or in ancestor paths
+    Example of using PathHierarchyTokenizerFactory for ancestor queries.
+    Since Lucene 10 produces sequential tokens, we need to store paths at 
index time
+    to match against sequential query tokens properly.
   -->
   <fieldType name="ancestor_path" class="solr.TextField">
     <analyzer type="index">
-      <tokenizer class="solr.KeywordTokenizerFactory"/>
+      <tokenizer class="solr.PathHierarchyTokenizerFactory" delimiter="/"/>
     </analyzer>
     <analyzer type="query">
       <tokenizer class="solr.PathHierarchyTokenizerFactory" delimiter="/"/>

Review Comment:
   minor/subjective: could combine the two `analyzer` elements since they are 
now identical



-- 
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: issues-unsubscr...@solr.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to