You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Michael Park (JIRA)" <ji...@apache.org> on 2016/10/14 22:19:20 UTC
[jira] [Updated] (MESOS-4566) Avoid unnecessary temporary
`std::string` constructions and copies in `jsonify`.
[ https://issues.apache.org/jira/browse/MESOS-4566?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Michael Park updated MESOS-4566:
--------------------------------
Description: A few of the critical code paths in {{jsonify}} involve unnecessary temporary string construction and copies (inherited from the {{JSON::*}}). For example, {{strings::trim}} is used to remove trailing 0s from printing {{double}}. We print {{double}} a lot, and therefore constructing a temporary {{std::string}} on printing of every double is extremely costly. This ticket captures the work involved in avoiding them. (was: A few of the critical code paths in {{jsonify}} involve unnecessary temporary string construction and copies (inherited from the {{JSON::*}}). For example, {{strings::trim}} is used to remove trailing 0s from printing {{double}}s. We print {{double}}s a lot, and therefore constructing a temporary {{std::string}} on printing of every double is extremely costly. This ticket captures the work involved in avoiding them.)
> Avoid unnecessary temporary `std::string` constructions and copies in `jsonify`.
> --------------------------------------------------------------------------------
>
> Key: MESOS-4566
> URL: https://issues.apache.org/jira/browse/MESOS-4566
> Project: Mesos
> Issue Type: Improvement
> Components: stout
> Reporter: Michael Park
> Assignee: Michael Park
> Labels: mesosphere
> Fix For: 0.24.2, 0.25.1, 0.26.1, 0.27.1, 0.28.0
>
>
> A few of the critical code paths in {{jsonify}} involve unnecessary temporary string construction and copies (inherited from the {{JSON::*}}). For example, {{strings::trim}} is used to remove trailing 0s from printing {{double}}. We print {{double}} a lot, and therefore constructing a temporary {{std::string}} on printing of every double is extremely costly. This ticket captures the work involved in avoiding them.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)