Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-511

copy method in LogicalAggregate not copying the indicator value properly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      The copy method in LogicalAggregate does not take the value of the indicator boolean from the input parameters, but from the object itself.

        Activity

        Hide
        jcamachorodriguez Jesus Camacho Rodriguez added a comment -
        Show
        jcamachorodriguez Jesus Camacho Rodriguez added a comment - Thanks Vladimir Sitnikov !
        Show
        vladimirsitnikov Vladimir Sitnikov added a comment - Looks good. Committed in https://git-wip-us.apache.org/repos/asf?p=incubator-calcite.git;a=commitdiff;h=155d24f1aa2b0733e402aca6c2b0ba1ed1ef456b
        Hide
        jcamachorodriguez Jesus Camacho Rodriguez added a comment -

        Julian Hyde, I created a pull request with the tiny fix for this. Could you check it (it is important to get CALCITE-461 right). Thanks!

        Show
        jcamachorodriguez Jesus Camacho Rodriguez added a comment - Julian Hyde , I created a pull request with the tiny fix for this. Could you check it (it is important to get CALCITE-461 right). Thanks!

          People

          • Assignee:
            jcamachorodriguez Jesus Camacho Rodriguez
            Reporter:
            jcamachorodriguez Jesus Camacho Rodriguez
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development