Oliver Eilhard
Oliver Eilhard
As mentioned [here](https://github.com/elastic/elasticsearch/pull/78531/files), it needs to be renamed to `shard_stats` (not `shards_stats` as in my PR).
Thanks for the repro.
Although I agree that `gt` and `gte` are more intuitive, the old way of doing things are still [alive and kicking](https://github.com/elastic/elasticsearch/blob/7.4/server/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java#L321-L341). I won't merge this PR in its current form...
I've checked 6.8.3 and 7.4.0, and both seem to be able to parse `gt` and `gte` correctly, so I think we can move forward with this change. Both seem to...
@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...
Does your program stop after the flush? If so, does adding a delay of, say, 5 seconds at the end of `main` help in any way?
@hastinc I treated this PR as a "nice-to-have" mainly because the on-the-wire format [still uses the old syntax](https://github.com/elastic/elasticsearch/blob/v7.12.1/server/src/main/java/org/elasticsearch/index/query/RangeQueryBuilder.java#L308-L328) even in the current version of ES. In other words: While the...
Yeah, it's a nice to have. But yeah, it's missing.
Fixing this is probably a breaking change. Moving to 8.x.
Yes, the specified decoder is just used for the direct responses to service calls. I have to review if there is a simply change.