Disable lowercase convention

[jnus]

Hi

Is it possible to disable the lowercase convention on all properties, when saving new documents?

Thanks
/Jasper

[damienbod]

Hi Jasper

At present, I have fixed all the document properties to lowercase values in Elasticsearch. If it would help you , I could change this to allow case sensitive properties in the Elasticsearch documents.

Greetings Damien

[jnus]

That would definitely help!! - Currently migrating logs from mssql to elasticsearch with existing case sensitive properties created my Serilog. ElasticsearchCRUD is just so much easier to use compared to NEST ;).

Thanks in advance! J.

[damienbod]

Hi Jasper

I'll make a release in August with this change. I'll make it so that the default config is lowercase and add a code configuration property which can remove this and use case sensitive properties.

Or do you have a better idea?

greetings Damien

[jnus]

What's the argument for defaulting to lower case? It seems a bit invasive to change the document format, when persisting to ES, merely by convention. I suspect you have implemented this, due to ES CS nature. However from a developers point of view, I would switch the behavior: by default no manipulation and a configuration to enable the lower casing of properties behavior. Just my 2 cents :)

Sounds great with a pending release in Aug.! Give me a heads up, if you can't find the time and I might do a PR for whichever strategy makes most sense.

Thanks J.

[damienbod]

Hi Jasper

Makes sense, I was just thinking of the existing clients who would need to change this for existing
ES stores when updating.

If you have time to do a PR, it would be great, I can't do anything until the start of August and I'll implement this then. I also wanted to add some of the changes/new features from ES 1.6.

Greetings Damien

[damienbod]

Thanks for this

I'll merge this next week and create a NuGet package,

Greetings Damien