> On Oct. 29, 2012, 10:38 a.m., Sijie Guo wrote: > > bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java, > > line 92 > > <https://reviews.apache.org/r/7733/diff/1/?file=179878#file179878line92> > > > > do we need to release semaphore when interrupted, since sendReadTo > > would acquire semaphore.
InterruptedException will only occur when trying to acquire the semaphore. If it does, it's safe to assume we didnt acquire it. - Ivan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7733/#review12869 ----------------------------------------------------------- On Oct. 25, 2012, 3:50 p.m., Ivan Kelly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7733/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2012, 3:50 p.m.) > > > Review request for bookkeeper. > > > Description > ------- > > The code to handle the state of a single entry read request is scattered all > over PendingReadOp. Some even leaks into LedgerEntry. This jira is to > refactor this code into one place to make speculative reads easier to > implement. > > > This addresses bug BOOKKEEPER-444. > https://issues.apache.org/jira/browse/BOOKKEEPER-444 > > > Diffs > ----- > > > bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java > 1f4547e > > bookkeeper-server/src/main/java/org/apache/bookkeeper/client/PendingReadOp.java > c8e814c > > Diff: https://reviews.apache.org/r/7733/diff/ > > > Testing > ------- > > > Thanks, > > Ivan Kelly > >
