mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <neil.con...@gmail.com>
Subject Review Request 48694: Optimized `UUID::fromString` and `UUID::toString` in stout.
Date Tue, 14 Jun 2016 14:56:59 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48694/
-----------------------------------------------------------

Review request for mesos and Anand Mazumdar.


Repository: mesos


Description
-------

Rather than using `std::istringstream` and `std::ostringstream`,
instead use the facilities provided by Boost's UUID for input
and output respectively. This improves a simple benchmark that
uses `fromString` and `toString` by ~7x.


Diffs
-----

  3rdparty/stout/include/stout/uuid.hpp cde3bdbbdcc428410c83a7724389056f7aaf63a1 

Diff: https://reviews.apache.org/r/48694/diff/


Testing
-------

make check

Benchmark is here: https://gist.github.com/anonymous/138d0333656871bddc2dfaa61fbf033e -- happy
to submit it as an RR as well (with cleanup), but I didn't think it was warranted.


Thanks,

Neil Conway


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message