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

Case sensitive property name configuration #132

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Case sensitive property name configuration #132

wants to merge 1 commit into from

Conversation

jnus
Copy link

@jnus jnus commented Jul 22, 2015

Changed default mapping of property names to case sensitive. Can be configured in ElasticsearchSerializerConfiguration ctor to enable lower case stategy on all property names. Implementation is as non-invasive as possible with the use of an extension method.

Example: configure lower case strategy

            return new ElasticsearchContext(
                                "http://localhost:9200/",
                                new ElasticsearchSerializerConfiguration(
                                                        CreateMappingResolver(date),
                                                        saveChildObjectsAsWellAsParent:true,
                                                        processChildDocumentsAsSeparateChildIndex:false,
                                                        userDefinedRouting:false,
                                                        mapToLowerCase:true)
                );

Example: configure case sensitive strategy (default)

            return new ElasticsearchContext(
                                "http://localhost:9200/",
                                new ElasticsearchSerializerConfiguration(
                                                        CreateMappingResolver(date))
                );

…igured in ElasticsearchSerializerConfiguration ctor to enable lower case stategy on all property names
@damienbod
Copy link
Owner

Hi Jasper, I am testing the pull branch and have noticed that the branch has a build problem and also that the Integration tests do not run because Elasticsearch indices have to be lowercase.

Greetings Damien

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants