Re: Issues in ZK event handling logic for BookiesChanged event

2017-03-07 Thread Charan Reddy G
*Issue 1:* *Matteo's Response:* *"The way I understand this is that you might get less notifications than* *the number of changes on the z-node, but when you set the watch, it shouldbe tied to the z-node version that you have already read, so you should beget notified for any change after

Re: SSL and Netty 4/OpenSSL news

2017-03-07 Thread Venkateswara Rao Jujjuri
On Tue, Mar 7, 2017 at 10:53 AM, Sijie Guo wrote: > On Mon, Mar 6, 2017 at 11:33 PM, Enrico Olivelli > wrote: > > > Sijie, > > Most of the upcoming patches from Yahoo will need Netty 4, and at > > Salesforce are working on Netty 4 + SSL, so the idea at the last meeting > > was to get Netty 4.1 i

Build failed in Jenkins: bookkeeper-master-git-pullrequest #255

2017-03-07 Thread Apache Jenkins Server
See Changes: [govindappumenon] [BOOKKEEPER-1009] Use multiple journals in bookie -- [...truncated 29.43 KB...] Tests run: 20, Failures: 0, Errors: 0, Skipp

Re: SSL and Netty 4/OpenSSL news

2017-03-07 Thread Sijie Guo
On Mon, Mar 6, 2017 at 11:33 PM, Enrico Olivelli wrote: > Sijie, > Most of the upcoming patches from Yahoo will need Netty 4, and at > Salesforce are working on Netty 4 + SSL, so the idea at the last meeting > was to get Netty 4.1 in in 4.5 > I think that most of the problem is that you local for

Re: Issues in ZK event handling logic for BookiesChanged event

2017-03-07 Thread Sijie Guo
On Tue, Mar 7, 2017 at 10:10 AM, Charan Reddy G wrote: > Hey, > > *Issue 1:* > > I was going through the code to understand how BookiesChanged event is > handled for AuditTask (in Auditor, BookkeeperAdmin and BookieWatcher > classes). In BookieWatcher.notifyBookiesChanged they are making > getChi

Re: Issues in ZK event handling logic for BookiesChanged event

2017-03-07 Thread Matteo Merli
On Tue, Mar 7, 2017 at 10:10 AM Charan Reddy G wrote: > But due to the limitation of ZK (which is well documented thing in ZK > https://zookeeper.apache.org/doc/r3.4.5/zookeeperProgrammers.html#ch_ > zkWatches), "Because watches are one time triggers and there is latency > between getting the eve

Issues in ZK event handling logic for BookiesChanged event

2017-03-07 Thread Charan Reddy G
Hey, *Issue 1:* I was going through the code to understand how BookiesChanged event is handled for AuditTask (in Auditor, BookkeeperAdmin and BookieWatcher classes). In BookieWatcher.notifyBookiesChanged they are making getChildren call on "bookieRegistrationPath" and setting watcher (which liste

Build failed in Jenkins: bookkeeper-master #1679

2017-03-07 Thread Apache Jenkins Server
See -- [...truncated 21.77 KB...] [INFO] Changes detected - recompiling the module! [INFO] Compiling 114 source files to