Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-43034

Special values handling is inconsistent in DoubleDoubleSummation

    • Fully Compatible
    • ALL
    • v4.2, v4.0, v3.6
    • Hide
      TEST(Summation, ConvertSpecialToDecimal) {
          constexpr double infinity = std::numeric_limits<double>::infinity();
          DoubleDoubleSummation sum;
          
          sum.addDouble(infinity);    
          ASSERT_EQUALS(infinity, sum.getDouble()); // ok
          ASSERT_TRUE(sum.getDecimal().isInfinite()); // <-- fail here, it's nan
          ASSERT_FALSE(sum.getDecimal().isNaN()); // <!-- fail here, it is actually nan
          
          sum.addDouble(1); // now once we add something to sum, everything is ok
          ASSERT_EQUALS(infinity, sum.getDouble()); // ok, still infinity
          ASSERT_TRUE(sum.getDecimal().isInfinite()); // ok, now it's infinity
          ASSERT_FALSE(sum.getDecimal().isNaN()); // ok, not nan
      }
      
      Show
      TEST(Summation, ConvertSpecialToDecimal) { constexpr double infinity = std::numeric_limits< double >::infinity(); DoubleDoubleSummation sum; sum.addDouble(infinity); ASSERT_EQUALS(infinity, sum.getDouble()); // ok ASSERT_TRUE(sum.getDecimal().isInfinite()); // <-- fail here, it's nan ASSERT_FALSE(sum.getDecimal().isNaN()); // <!-- fail here, it is actually nan sum.addDouble(1); // now once we add something to sum, everything is ok ASSERT_EQUALS(infinity, sum.getDouble()); // ok, still infinity ASSERT_TRUE(sum.getDecimal().isInfinite()); // ok, now it's infinity ASSERT_FALSE(sum.getDecimal().isNaN()); // ok, not nan }
    • Execution Team 2019-10-07
    • 12

      If the result of DoubleDoubleSummation is Infinity, it may return different special values depending on whether the result is converted to double or decimal.

            Assignee:
            geert.bosch@mongodb.com Geert Bosch
            Reporter:
            anton.korshunov@mongodb.com Anton Korshunov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: