Uploaded image for project: 'Mesos'
  1. Mesos
  2. MESOS-4235

JSON generation performance improvement

    Details

    • Epic Name:
      json-perf

      Description

      This is an epic which evolved from MESOS-2353. As mentioned in the description of MESOS-2353, most of the work is spent performing memory allocation/deallocation. Some preliminary efforts have been made such as calling reserve for JSON::Array. There are still plenty of dynamic allocations being made especially from instances of JSON::Object which hold a std::map as a member.

      The current approach being adopted is to introduce a jsonify function which by-passes these unnecessary dynamic allocations and copying, and to simply hold references to the underlying objects.

      We plan to first introduce the jsonify function to stout, and update master's state endpoint, then proceed to update the rest of the system.

        Issue Links

          Issues in Epic

            Activity

            Hide
            mcypark Michael Park added a comment - - edited

            There are still follow-up work to be done in regards to migrating rest of the endpoints, but the scope of this ticket has been met. Refer to MESOS-4732

            Show
            mcypark Michael Park added a comment - - edited There are still follow-up work to be done in regards to migrating rest of the endpoints, but the scope of this ticket has been met. Refer to MESOS-4732
            Hide
            mcypark Michael Park added a comment -

            Cong Wang There's some more tickets to be resolved that hadn't been filed yet. Sorry about the confusion, we basically need to migrate the rest of the endpoints beyond /state and finalize through the pattern.

            Show
            mcypark Michael Park added a comment - Cong Wang There's some more tickets to be resolved that hadn't been filed yet. Sorry about the confusion, we basically need to migrate the rest of the endpoints beyond /state and finalize through the pattern.
            Hide
            wangcong Cong Wang added a comment -

            Hi, Michael Park
            Since all the subtickets are resolved, I assume this one is resolved too in the latest code base?

            Show
            wangcong Cong Wang added a comment - Hi, Michael Park Since all the subtickets are resolved, I assume this one is resolved too in the latest code base?

              People

              • Assignee:
                mcypark Michael Park
                Reporter:
                mcypark Michael Park
                Shepherd:
                Benjamin Hindman
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development