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

Projects
None yet
3 participants
@kzwang
Copy link
Contributor

commented Mar 26, 2014

added UpperCaseFilter which was added in Lucene 4.7

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

@martijnvg

This comment has been minimized.

Copy link
Member

commented Mar 26, 2014

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

@kzwang

This comment has been minimized.

Copy link
Contributor Author

commented Mar 26, 2014

@martijnvg added

@martijnvg

This comment has been minimized.

Copy link
Member

commented Mar 26, 2014

@kzwang Thanks, I will pull it in soon.

martijnvg added a commit that referenced this pull request Mar 26, 2014

@martijnvg martijnvg closed this in 374b633 Mar 26, 2014

@martijnvg

This comment has been minimized.

Copy link
Member

commented Mar 26, 2014

@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 kzwang:feature/upper_case_filter branch Mar 26, 2014

s1monw added a commit that referenced this pull request May 18, 2014

Add missing coming tag
Relates to #6188
Relates to #5539

s1monw added a commit that referenced this pull request May 18, 2014

Add missing coming tag
Relates to #6188
Relates to #5539

@clintongormley clintongormley changed the title add uppercase token filter Add uppercase token filter Jun 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.