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

HighlightBuilder should be consistent with SearchContextHighlight.Field options #3435

Closed
s1monw opened this Issue Aug 2, 2013 · 1 comment

Comments

Projects
None yet
1 participant
@s1monw
Contributor

s1monw commented Aug 2, 2013

Today the HighlighBuilder doesn't expose all the options that are possible via the REST API. We need to expose the missing options via the Java API for consistency as well.

@ghost ghost assigned s1monw Aug 2, 2013

@s1monw

This comment has been minimized.

Show comment
Hide comment
@s1monw

s1monw Aug 2, 2013

Contributor

This PR fixes this issue #3275

Contributor

s1monw commented Aug 2, 2013

This PR fixes this issue #3275

@s1monw s1monw closed this in f2168d3 Aug 2, 2013

s1monw added a commit that referenced this issue Aug 2, 2013

Make (HighlightBuilder|SearchContextHighlight).Field consistent
Update HighlightBuilder.Field API, it should allow for the same API
as SearchConstextHighlight.Field. In other words, what is possible
to setup using DSL in highlighting at the field level is also
possible via the Java API.

Closes #3435

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Make (HighlightBuilder|SearchContextHighlight).Field consistent
Update HighlightBuilder.Field API, it should allow for the same API
as SearchConstextHighlight.Field. In other words, what is possible
to setup using DSL in highlighting at the field level is also
possible via the Java API.

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