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

Remove unused includes

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Trivial - P5 Trivial - P5
    • 8.1.0-rc0
    • Affects Version/s: None
    • Component/s: Query Execution
    • None
    • Query Execution
    • Fully Compatible
    • QE 2024-12-09

      Remove unused includes from various files, as these headers are not needed to be included.

      $ git diff --stat
       src/mongo/db/query/get_executor.cpp                            | 49 -------------------------------------------------
       src/mongo/db/query/get_executor.h                              | 11 -----------
       src/mongo/s/commands/query_cmd/cluster_find_and_modify_cmd.cpp |  9 ---------
       src/mongo/s/commands/query_cmd/cluster_find_and_modify_cmd.h   | 28 ----------------------------
       src/mongo/s/commands/strategy.cpp                              |  5 -----
       src/mongo/s/commands/strategy.h                                |  2 --
       6 files changed, 104 deletions(-)

      I noticed these includes are not necessary while working on other PRs, but I did not want to extend the scope of the other PRs with unrelated header removal. Thus I am batching the removal in this PR here.

      The unused includes were indicated by the VS code clang extension. The code still compiles when removing them.

            Assignee:
            jan.steemann@mongodb.com Jan Steemann
            Reporter:
            jan.steemann@mongodb.com Jan Steemann
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: