I changed the signature of executeQuery() from 'void' to 'Boolean' so as to
control the chain of responsibilities's flow. For instance, when
ElasticSearchFinder succeeds, I can stop the propagation by returning
'true' and to let the MagamFinder proceed by returning 'false'. Let me know
if there is a problem with this logic.


On Tue, Jan 29, 2013 at 8:13 PM, Cedric Thiebault
<notifications@github.com>wrote:

> In
> opal-search/src/main/java/org/obiba/opal/web/math/support/CategoricalSummaryStatsMagmaFinder.java:
>
> >      }
> >      builder.setMode(mode).setN(freq.getSumFreq());
> > -    return TimestampedResponses.ok(getValueTable(), SummaryStatisticsDto.newBuilder().setResource(getVariable().getName()).setExtension(CategoricalSummaryDto.categorical, builder.build()).build()).build();
> > +    result.setValue(builder.build());
> > +
> > +    return true;
>
> Return true instead of Response?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/obiba/opal/pull/30/files#r2819718>.
>
>