I can help fixing that bug, should I file a JIRA?

BTW, I will be posting an updated PR for this today , with latest merged
upstream and added tests

On Jun 27, 2017 4:56 AM, "Julian Hyde" <[email protected]> wrote:

> It looks as if you’re running into a bug in HepPlanner. I may have run
> into something similar when I was working on https://issues.apache.org/
> jira/browse/CALCITE-873 <https://issues.apache.org/jira/browse/CALCITE-873
> >.
>
> > On Jun 25, 2017, at 8:13 PM, Atri Sharma <[email protected]> wrote:
> >
> > Hi All,
> >
> > Please advise.
> >
> > Regards,
> >
> > Atri
> >
> > On Jun 25, 2017 11:25 PM, "Atri Sharma" <[email protected]> wrote:
> >
> >> Debugging a bit further, I realised that the code flow is stuck in
> >> applyRules. In the following code:
> >>
> >> while (iter.hasNext()) {
> >>  HepRelVertex vertex = iter.next();
> >>  for (RelOptRule rule : rules) {
> >>    HepRelVertex newVertex =
> >>        applyRule(rule, vertex, forceConversions);
> >>
> >> vertex is constantly the top level LogicalSort node, and not moving
> >> down the plan.
> >>
> >> Is there something that I am missing?
> >>
> >> Regards,
> >>
> >> Atri
> >>
> >> On Sun, Jun 25, 2017 at 2:30 PM, Atri Sharma <[email protected]>
> wrote:
> >>> Hi All,
> >>>
> >>> I am working on removing redundant sorts and am stuck when
> >>> transforming the call to replace older Sort node with the reduced one.
> >>>
> >>> The code works like following:
> >>>
> >>> final RelCollation newCollation =  RelCollations.of(collationsList);
> >>>
> >>> Sort result = sort.copy(sort.getTraitSet(), sort.getInput(),
> >> newCollation,
> >>>        sort.offset, sort.fetch);
> >>>
> >>> call.transformTo(result);
> >>>
> >>> The trnasformTo call is stuck forever.
> >>>
> >>> The query is:
> >>>
> >>> select count(*) as c\n"
> >>>        + "from sales.emp\n"
> >>>        + "where deptno = 10\n"
> >>>        + "group by deptno, sal\n"
> >>>        + "order by deptno, sal";
> >>>
> >>>
> >>> Please advise
> >>>
> >>> Regards,
> >>>
> >>> Atri
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>> --
> >>> Regards,
> >>>
> >>> Atri
> >>> l'apprenant
> >>
> >>
> >>
> >> --
> >> Regards,
> >>
> >> Atri
> >> l'apprenant
> >>
>
>

Reply via email to