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

Primitive.DOUBLE.min should be large and negative

        Details

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

          Description

          Kylin user reported max() of negative doubles does not return correct result. https://issues.apache.org/jira/browse/KYLIN-751

          The root cause is the initial value of calcite aggregator is not a negative double. In Primitive.java, it took Double.MIN_VALUE as the smallest double value but that actually is not a negative number according to javadoc.

            Issue Links

            is related to

            Bug - A problem which impairs or prevents the functions of the product. KYLIN-751 Max on negative double values is not working

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/b812e045.

              Show
              julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/b812e045 .
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Resolved in release 1.3.0-incubating (2015-05-30).

              Show
              julianhyde Julian Hyde added a comment - Resolved in release 1.3.0-incubating (2015-05-30).

                People

                • Assignee:
                  julianhyde Julian Hyde
                  Reporter:
                  liyang.gmt8@gmail.com liyang
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development