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 limit to total number of fields in mapping #17203

Closed
wants to merge 0 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@yanjunh
Contributor

yanjunh commented Mar 19, 2016

Closes #13547
This is to prevent mapping explosion when dynamic keys such as UUID are used as field names. index.mapping.total_fields.limit specifies the total number of fields an index can have. An exception will be thrown when the limit is reached. The default limit is 0 which means no limit. This setting is runtime adjustable.

@jpountz

View changes

Show outdated Hide outdated core/src/main/java/org/elasticsearch/index/mapper/MapperService.java
@jpountz

This comment has been minimized.

Show comment
Hide comment
@jpountz

jpountz Mar 22, 2016

Contributor

Thanks @yanjunh. Would you mind adding some docs a test that enables the limit and makes sure that you get an exception when trying to add more fields? (like #15989)

Contributor

jpountz commented Mar 22, 2016

Thanks @yanjunh. Would you mind adding some docs a test that enables the limit and makes sure that you get an exception when trying to add more fields? (like #15989)

@jpountz jpountz self-assigned this Mar 22, 2016

@yanjunh

This comment has been minimized.

Show comment
Hide comment
@yanjunh

yanjunh Mar 24, 2016

Contributor

@jpountz Sure. I will take a look this weekend. thanks.

Contributor

yanjunh commented Mar 24, 2016

@jpountz Sure. I will take a look this weekend. thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment