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

parallelize clang-tidy task

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.9.0
    • Component/s: Build
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Sprint:
      Execution Team 2020-11-02

      Description

      Currently, it appears that the clang-tidy task invokes one command to scan the entire codebase sequentially, and thus takes multiple hours to complete. To get better use out of clang-tidy and encourage its use in a popular builder, we should change it to run individually on each source file, in a parallel fashion.

        Attachments

          Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-27984 Add clang-tidy run in addition to lint

          • Major - P3 - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-49800 add use-after-move rule to clang-tidy evergreen builder

          • Major - P3 - Major loss of function.
          • Closed
          related to

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-51240 Add clang tidy rules to prevent const return by value and (in declarations) const pass by value

          • Major - P3 - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-54210 add clang-tidy to commit queue tasks

          • Major - P3 - Major loss of function.
          • Closed

            Activity

              People

              Assignee:
              benety.goh Benety Goh
              Reporter:
              milkie Eric Milkie
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: