-
Notifications
You must be signed in to change notification settings - Fork 24.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add PathHierarchy type back to path_hierarchy tokenizer for backward compatibility with 1.x #15785
Conversation
Is there a way that we could only support it for indices created before 2.0 to ensure that new indices will use the lowercase version? |
Maybe we could register a subclass of PathHierarchyTokenizerFactory whose constructor fails if the version in the index settings is >= 2.0? The error message could even point to the replacement to use? |
Hmm after all we will have to do it for others analysis components (eg. edgeNGram) so let's push it as-is and work on a migration path as part of #8988 |
3999895
to
3cac1ae
Compare
…compatibility with 1.x Closes elastic#15785
3cac1ae
to
de6dfe1
Compare
Add PathHierarchy type back to path_hierarchy tokenizer for backward compatibility with 1.x
…compatibility with 1.x Closes #15785
…compatibility with 1.x Closes #15785
…compatibility with 1.x Closes #15785
…compatibility with 1.x Closes #15785
@ywelsch this went into 2.2 as well can you label it accordingly? |
@s1monw done |
Relates to #15756