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

Consider refactoring the collation logic of IDHACK solution for delete command into ParsedDelete

    • Type: Icon: Improvement Improvement
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Labels:
      None
    • Query Execution

      This is a refactoring ticket.

      Currently we don't go through the normal query parsing logic of ParsedDelete for IDHACK solution for delete command inside ParsedDelete::parseRequest().

      Instead, we manually compose IDHACK plan inside getExecutorDelete(). As part of this process, we figure out the collator to use.

      Given that recently timeseries arbitrary updates/deletes project added a collator resolution logic, we should be able to reuse ParsedDelete's expCtx's collator for IDHACK plan too.

      Probably we may want to IDHACK eligibility check into ParsedDelete too and add isIDHackEligible() method and only do the final step like creating IDHackStage and generate the IDHACK plan inside getExecutorDelete()

            Assignee:
            colin.stolley@mongodb.com Colin Stolley
            Reporter:
            yoonsoo.kim@mongodb.com Yoon Soo Kim
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: