Re: Review Request 17353: Add a stat that tracks the number of threads waiting for the storage lock.

2014-01-25 Thread Bill Farner
On Jan. 25, 2014, 6:58 p.m., Joe Smith wrote: src/main/java/org/apache/aurora/scheduler/storage/ReadWriteLockManager.java, line 116 https://reviews.apache.org/r/17353/diff/1/?file=450927#file450927line116 potentially naive thought- might it be interesting to also have a way to

Re: Review Request 17353: Add a stat that tracks the number of threads waiting for the storage lock.

2014-01-25 Thread Joe Smith
On Jan. 25, 2014, 10:58 a.m., Joe Smith wrote: src/main/java/org/apache/aurora/scheduler/storage/ReadWriteLockManager.java, line 116 https://reviews.apache.org/r/17353/diff/1/?file=450927#file450927line116 potentially naive thought- might it be interesting to also have a way

Re: Review Request 17332: Add a noun supporting operations on roles

2014-01-25 Thread Mark Chu-Carroll
Interesting idea... I just thought of quota as being a natural property of the role. Are there are quota verbs that would make sense? Maybe an automated quota request could be a verb? On Fri, Jan 24, 2014 at 5:35 PM, Kevin Sweeney kevi...@apache.org wrote: This is an automatically

Re: Review Request 17185: Merged all of the open clientv2 reviews into one unified change.

2014-01-25 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17185/#review32577 --- k - sorry - this review has been sitting completed in my

Review Request 17359: Avoid using protobuf equals().

2014-01-25 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17359/ --- Review request for Aurora and Joe Smith. Repository: aurora Description

Re: Review Request 17359: Avoid using protobuf equals().

2014-01-25 Thread Jonathan Boulle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17359/#review32787 --- Ship it! Ship It! - Jonathan Boulle On Jan. 26, 2014, 12:35