Skip to content
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 uppercase token filter #5539

Closed
wants to merge 1 commit into from

Conversation

kzwang
Copy link
Contributor

@kzwang kzwang commented Mar 26, 2014

added UpperCaseFilter which was added in Lucene 4.7

https://issues.apache.org/jira/browse/LUCENE-5369

@martijnvg
Copy link
Member

+1 This change looks good to, can you add: 'closes #5539' to the commit message?

@kzwang
Copy link
Contributor Author

kzwang commented Mar 26, 2014

@martijnvg added

@martijnvg
Copy link
Member

@kzwang Thanks, I will pull it in soon.

martijnvg pushed a commit that referenced this pull request Mar 26, 2014
@martijnvg martijnvg closed this in 374b633 Mar 26, 2014
@martijnvg
Copy link
Member

@kevinkluge I made a small change to it, in PreBuiltTokenFilters.java I also added the upper_case token filter, so it globally available by default, just like lowercase token filter. This will remove the need to define an uppercase token filter specifically in the settings of an index.

Thanks for your contribution!

@kzwang kzwang deleted the feature/upper_case_filter branch March 26, 2014 08:39
s1monw added a commit that referenced this pull request May 18, 2014
Relates to #6188
Relates to #5539
s1monw added a commit that referenced this pull request May 18, 2014
Relates to #6188
Relates to #5539
@clintongormley clintongormley changed the title add uppercase token filter Add uppercase token filter Jun 6, 2015
@clintongormley clintongormley added the :Search/Analysis How text is split into tokens label Jun 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants