mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <bmah...@apache.org>
Subject Re: Review Request 58828: Replaced std::set with hashset for framework roles.
Date Fri, 28 Apr 2017 23:15:38 GMT


> On April 28, 2017, 6:30 a.m., Michael Park wrote:
> > It's true that we don't use `std::set` for its ordering in the general case,
> > however, I think the ordering does help produce easier-to-read error messages.
> > 
> > 
> > What do you think of preserving that property?

Yes you're right, I forgot that was the original reason I suggested std::set, given we log
it. My intention with this change was to order it when we need to (before logging it), but
seems it is not worth the trouble, I'll drop this.


- Benjamin


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


On April 28, 2017, 12:43 a.m., Benjamin Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58828/
> -----------------------------------------------------------
> 
> (Updated April 28, 2017, 12:43 a.m.)
> 
> 
> Review request for mesos and Michael Park.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Since we do not need ordering, this updates the framework roles to
> be stored using hashset instead of std::set.
> 
> 
> Diffs
> -----
> 
>   src/common/protobuf_utils.hpp be2325f05b81b847fa592eff65175cbc99764fd6 
>   src/common/protobuf_utils.cpp 3fcaf786b29a00f003c10b0f1614a2c7eddc725d 
>   src/master/allocator/mesos/hierarchical.hpp 219f508dacb3b372c12da3f15e07a3bf5f27e6e6

>   src/master/allocator/mesos/hierarchical.cpp 984a0a4e2671ee7bb2a3515849342f49f2c4e3aa

>   src/master/master.hpp d537933d0b467a6f9996951c601b31338bb9d034 
>   src/master/master.cpp a0cb505a1300637f42fa8062c57deb1590779dd4 
>   src/slave/slave.cpp 8b8078dbb656e9db2efa53cc4ec5bed2cc01d49a 
>   src/tests/protobuf_utils_tests.cpp 5239182812835b93a28e85146b2df2b20ae77328 
> 
> 
> Diff: https://reviews.apache.org/r/58828/diff/1/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Benjamin Mahler
> 
>


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