Skip to content

Make strategy optional in GeoShapeQueryBuilder readFrom and writeTo#13963

Merged
javanna merged 1 commit intoelastic:masterfrom
javanna:fix/geo_shape_strategy_optional
Oct 6, 2015
Merged

Make strategy optional in GeoShapeQueryBuilder readFrom and writeTo#13963
javanna merged 1 commit intoelastic:masterfrom
javanna:fix/geo_shape_strategy_optional

Conversation

@javanna
Copy link
Copy Markdown
Contributor

@javanna javanna commented Oct 6, 2015

The field is optional everywhere else but in the serialization methods, which causes problems. Also expanded tests so that they can catch this type of problem.

The field is optional everywhere else but in the serialization methods, which causes problems. Also expanded tests so that they can catch this type of problem.

Closes elastic#13963
@colings86
Copy link
Copy Markdown
Contributor

LGTM

javanna added a commit that referenced this pull request Oct 6, 2015
Make strategy optional in GeoShapeQueryBuilder readFrom and writeTo
@javanna javanna merged commit 84b748c into elastic:master Oct 6, 2015
@clintongormley clintongormley added :Search/Search Search-related issues that do not fall into other categories and removed :Query DSL labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants