Nope. In this case, it will respond terminatedEarly=false even if noone
request it.

On Mon, Sep 3, 2018 at 9:09 AM zhenyuan wei <tins...@gmail.com> wrote:

> Yeah,got it~. So the QueryResult.segmentTerminatedEarly maybe a boolean,
> instead of Boolean,  is better, right?
>
> Mikhail Khludnev <m...@apache.org> 于2018年9月3日周一 下午1:36写道:
>
> > It's neither, it's on purpose. By default  result.segmentTerminatedEarly
> is
> > null, hence it doesn't appear in result output. see
> > ResponseBuilder.setResult(QueryResult).
> > So, if cmd requests early termination, it sets false by default, enabling
> > "false" output even it won't be the case. And later it might be flipped
> to
> > true.
> >
> >
> > On Mon, Sep 3, 2018 at 5:57 AM zhenyuan wei <tins...@gmail.com> wrote:
> >
> > > Hi all,
> > >     I saw the code like following:
> > >
> > > QueryResult result = new QueryResult();
> > >
> > >
> > >
> >
> cmd.setSegmentTerminateEarly(params.getBool(CommonParams.SEGMENT_TERMINATE_EARLY,
> > > CommonParams.SEGMENT_TERMINATE_EARLY_DEFAULT));
> > > if (cmd.getSegmentTerminateEarly()) {
> > >   result.setSegmentTerminatedEarly(Boolean.FALSE);
> > > }
> > >
> > > It says if request's param segmentTerminateEarly=true, which means
> search
> > > maybe terminated early within a segment,  then set
> > > result.setSegmentTerminatedEarly as false , this code is of a little
> > > confusion
> > > .
> > >
> >
> >
> > --
> > Sincerely yours
> > Mikhail Khludnev
> >
>


-- 
Sincerely yours
Mikhail Khludnev

Reply via email to