Re: Review Request 18240: Accumulate read/write timing in MemStorage for top-level operations only.

2014-02-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18240/#review35454 --- Ship it! Ship It!

Re: Review Request 18240: Accumulate read/write timing in MemStorage for top-level operations only.

2014-02-25 Thread Bill Farner
On Feb. 19, 2014, 4:33 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/storage/Storage.java, line 69 https://reviews.apache.org/r/18240/diff/1/?file=497126#file497126line69 typo Fixed. - Bill ---

Re: Review Request 18240: Accumulate read/write timing in MemStorage for top-level operations only.

2014-02-25 Thread Bill Farner
On Feb. 25, 2014, 8:35 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/storage/ReadWriteLockManager.java, line 113 https://reviews.apache.org/r/18240/diff/1/?file=497125#file497125line113 reads better statically imported (here and elsewhere). Done. - Bill

Re: Review Request 18240: Accumulate read/write timing in MemStorage for top-level operations only.

2014-02-25 Thread Bill Farner
On Feb. 25, 2014, 8:35 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/storage/ReadWriteLockManager.java, line 113 https://reviews.apache.org/r/18240/diff/1/?file=497125#file497125line113 reads better statically imported (here and elsewhere). Bill Farner