Range query: Follow latest DSL#1187
Range query: Follow latest DSL#1187and1truong wants to merge 1 commit intoolivere:release-branch.v7from
Conversation
458f527 to
0259048
Compare
0259048 to
4e28ea6
Compare
|
Although I agree that I won't merge this PR in its current form as it breaks existing clients that rely on |
|
I've checked 6.8.3 and 7.4.0, and both seem to be able to parse |
|
@olivere thanks for your great work, and I wonder why this pr is open for nearly a year after all check passed. IMO this is a really important change, and at least in ES 7.10, the old params are already deprecated. We really need this when using newer servers. Is there any more work need to do, or any help could be useful? |
|
@houz42 Thanks for the kind words. The reason is that there was just no need to rush. Even the Java code in 7.15 generates the range fields with the old syntax. See for youself. So in the end, it's just what happens on the wire. The actual outcome is the same. |
DSL for branch 7.x already changed, from/to/include_lower/include_upper are gone:
https://www.elastic.co/guide/en/elasticsearch/reference/7.0/query-dsl-range-query.html