Using a single 'Range' query inside a Query can cause a server side error

[kohner86]

Not sure if this is by design but if I define a query with a single QueryDescriptor inside of type Range in which the 'OnField' value is null ES throws a QueryParsingException since the emitted JSON in the query is empty due to no queries being present.

It seems that the nest library will automatically inject a match_all into the query if none are specified but in the above case the match_all is not added.

In the mean time I've been able to stop the issue from popping up by putting the Range query in a boolean query paired with a match_all.

[gmarz]

@kohner86 would you mind posting your code or an example which produces this error?

[kohner86]

Sure,
Here is a snippet which generates the error when minDate and maxDate are both null:
ISearchResponse authorStats = await esClient.SearchAsync(s =>
s.Take(0)
.Query(h =>
h.Filtered(w =>
w.Filter(f =>
f.Term(g => g.FeedId, feedId))
.Query(r =>
r.Range(x => x
.OnField(d => d.Service.Created)
.GreaterOrEquals(minDate)
.LowerOrEquals(maxDate)))))
.SortDescending(z => z.Service.Created)
.Take(_numToTake));

Which generates the following error:
nested: QueryParsingException[[default1] [_na] query malformed, no field after start_object]; }

[gmarz]

@kohner86

This was a bug with FilteredQuery, and not Range. I just pushed the above fix. Thanks for reporting this!

[BjBlazkowicz]

I'm also getting errors with the RangeQueryDescriptor when using OnField:

        var result = _client.Search<Movie>(
            x => x.Query(q => q
                .Range(
                    r => r
                        .GreaterOrEquals(DateTime.Now.AddSeconds(-secondsBack))
                        .OnField(f => f.LastSeen)
                        ))
                .From(from)
                .Take(take)
                .SortDescending(s => s.Viewers));

[gmarz]

@BjBlazkowicz What is the error you're getting?

[BjBlazkowicz]

Sorry gmarz, no exceptions and errors. My bad.

But I think we should be able to set the timezone in the Range query?

[gmarz]

@BjBlazkowicz sorry this fell under my radar since the issue was closed. Yup, you are totally right, time_zone was just added in ES 1.4. I opened #1158.