Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-21 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review62591 --- Ship it! Ship It! - Kevin Sweeney On Nov. 20, 2014, 11:43 a.m.,

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/ --- (Updated Nov. 20, 2014, 7:43 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-20 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review62406 --- Master (6d06d86) is red with this patch.

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-19 Thread Bill Farner
On Nov. 17, 2014, 7:20 p.m., Maxim Khutornenko wrote: I don't see ReadWriteLockManager removal in this diff. It was only used in mem storage AFAICT. Thanks, removed. - Bill --- This is an automatically generated e-mail. To reply,

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-19 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/ --- (Updated Nov. 19, 2014, 11:41 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review62283 --- This patch does not apply cleanly on master (4e52d00), do you need

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review62320 --- Ship it! Ship It! - Maxim Khutornenko On Nov. 19, 2014, 11:41

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-17 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review61769 --- I don't see ReadWriteLockManager removal in this diff. It was only

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/ --- (Updated Nov. 16, 2014, 10:43 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review61692 --- Master (8741cdb) is red with this patch.

Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-15 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs:

Re: Review Request 28097: Remove ReadWriteLock from MemStorage, remove Storage#weaklyConsistentRead.

2014-11-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28097/#review61652 --- Master (8741cdb) is red with this patch.