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

ElasticsearchString attribute appears to not work #154

Open
tbrettallen opened this issue Jun 28, 2016 · 6 comments
Open

ElasticsearchString attribute appears to not work #154

tbrettallen opened this issue Jun 28, 2016 · 6 comments

Comments

@tbrettallen
Copy link
Contributor

tbrettallen commented Jun 28, 2016

I've decorated properties in a few of my classes with [ElasticsearchString(Index = StringIndex.not_analyzed)] on several of my classes, and when they're saved to Elasticsearch, the created index still marks them as analyzed.

Attempted to debug this myself, but could not find where this is checked anywhere in ElasticsearchCRUD. Is this feature not yet implemented?

@damienbod
Copy link
Owner

should be, I'll have a look, which version are you using?

greetings Damien

@damienbod
Copy link
Owner

works with the preview version and Elastic version number": "2.3.1", I checked the mapping and its not analyzed.

Maybe an attribute property is no longer supported.

https://github.com/damienbod/ElasticsearchCRUD/blob/master/ElasticsearchCrudAspNet5/src/ElasticsearchGermanAnalyzer/ElasticsearchProvider.cs#L85

hope this helps

greetings Damien

@tbrettallen
Copy link
Contributor Author

tbrettallen commented Jul 5, 2016

Thanks! Elasticsearch 2.3.1 and was latest version of your code. It's a console application as I have an NServiceBus endpoint handling all elasticsearch updates.

That source file returns 404.

Which source files should I look at to reimplement this functionality?

I saw you've made changes since I've last looked at this. Was any of that related?

@damienbod
Copy link
Owner

damienbod commented Jul 5, 2016

sorry, I've updated to netstandard 1.4.

Usage:
https://github.com/damienbod/ElasticsearchCRUD/blob/master/src/ElasticsearchGermanAnalyzer/ElasticsearchProvider.cs#L85

The console app is a demo which uses this.

The changes I made are just porting the application to netstandard 1.4, so from version 2.3.3.1 onwards, only net461 or higher is supported.

I would like to support net451, but its too much effort testing for both.

Greetings Damien

@tbrettallen
Copy link
Contributor Author

I use 4.6.1 :)

I'll test the console app and if the non_analyzed option doesn't work still I'll use that to try to fix it. Thanks!

@damienbod
Copy link
Owner

Great, thanks

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

No branches or pull requests

2 participants