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

Investigate and fix up accumulator_test on v6.0

    • Type: Icon: Bug Bug
    • Resolution: Duplicate
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Integration
    • ALL

      It looks like a bunch of the assertions authored in SERVER-73317 are not passing on the v6.0 branch because there is some odd '0'-suffix and an extra array. Compare these two for an example:

      // Expected:
      {$topN: {n: {$const: "?"}, output: {HASH(output): "$HASH(output)", HASH(sortFields): ["$HASH(sortKey)"]}, sortBy: {HASH(sortKey): 1}}}
      // Actual:
      {$topN: {n: {$const: "?"}, output: {HASH(output): "$HASH(output)", HASH(sortFields0): "$HASH(sortKey)"}, sortBy: {HASH(sortKey): 1}}} 

      Nothing really stands out to me in the history of changes to the test file - it would have to be something between SERVER-73317 redaction for $function and $accumulator f7e235f (the original commit, where we know the tests pass) and SERVER-82368 In top/bottom accumulator map sort fields to object fields not array(the last commit to this file on v6.0). SERVER-60781 might be it?  But we may have to look at the changes to the non-test file as well to try to narrow it down.

            Assignee:
            alyssa.clark@mongodb.com Alyssa Clark
            Reporter:
            charlie.swanson@mongodb.com Charlie Swanson
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: