[
https://issues.apache.org/jira/browse/MESOS-4561?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15124658#comment-15124658
]
Michael Park commented on MESOS-4561:
-------------------------------------
The use of {{std::string::compare}} in the following files should be tackled
with this patch.
* {{3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/rmdir.hpp}}
* {{3rdparty/libprocess/3rdparty/stout/include/stout/windows/fs.hpp}}
> Investigate `std::string` to C string comparison.
> -------------------------------------------------
>
> Key: MESOS-4561
> URL: https://issues.apache.org/jira/browse/MESOS-4561
> Project: Mesos
> Issue Type: Bug
> Reporter: Michael Park
> Labels: mesosphere
>
> [~jvanremoortere] identified a large performance bottleneck in role
> validation logic and submitted a
> [fix|https://github.com/apache/mesos/commit/5e618ac2197e856f09511850bebe51fea71c1844]
> for the 0.27.0 release.
> However, templated versions of {{bool operator==(const std::string&, const
> char \*)}} and {{bool operator==(const char\*, const std::string&)}} are part
> of the standard library
> http://en.cppreference.com/w/cpp/string/basic_string/operator_cmp, and
> therefore should not have triggered temporary string constructions.
> The first thought was perhaps they don't happen to be implemented in
> {{libstdc++}}, but having looked at its implementation, I was able to find
> the definitions of these operators.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
