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

Refactor ExpressionContext to use designated initializers

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major - P3 Major - P3
    • None
    • None
    • None
    • Query Execution

    Description

      Currently ExpressionContext has multiple constructors that has multiple default values in their arguments.

      Multiple commands like find and distinct create the ExpressionContext from the existing constructors and pass multiple attributes that should have been default values

      Proposal is to use designated initilizer when constructing ExpressionContext and if constructors are necessary, then we could create a aggregate config struct and use it as the constructor parameter.

      The example usage would be like this:

       

      auto pool = ThreadPool({.min_threads = 1, .max_threads=10, .idle_time = Seconds(10)}) 

       

      Attachments

        Activity

          People

            backlog-query-execution Backlog - Query Execution
            denis.grebennicov@mongodb.com Denis Grebennicov
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated: