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

Analyzer def using search mapping #747

Merged
merged 3 commits into from Nov 13, 2017

Conversation

Projects
None yet
3 participants
@JiajingLiang
Contributor

JiajingLiang commented Oct 5, 2017

This is a separation of analyser definitions from entity class in order to provide more flexibility on search engine configuration. For example, replacing embedded Apache Lucene by ElasticSearch

@jamesagnew

This comment has been minimized.

Show comment
Hide comment
@jamesagnew

jamesagnew Oct 5, 2017

Owner

Wow, will review in depth a bit later but that is super cool!

Owner

jamesagnew commented Oct 5, 2017

Wow, will review in depth a bit later but that is super cool!

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Oct 5, 2017

Coverage Status

Coverage increased (+0.04%) to 81.736% when pulling 153c67f on honestica:analyzer-def-using-search-mapping into b412767 on jamesagnew:master.

coveralls commented Oct 5, 2017

Coverage Status

Coverage increased (+0.04%) to 81.736% when pulling 153c67f on honestica:analyzer-def-using-search-mapping into b412767 on jamesagnew:master.

Jiajing LIANG and others added some commits Oct 5, 2017

jamesagnew added a commit that referenced this pull request Nov 13, 2017

@jamesagnew jamesagnew merged commit 8f44af7 into jamesagnew:master Nov 13, 2017

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@jamesagnew jamesagnew referenced this pull request Nov 14, 2017

Merged

Add Spring Boot Starter #743

@JiajingLiang JiajingLiang deleted the honestica:analyzer-def-using-search-mapping branch Nov 27, 2017

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