[GitHub] bookkeeper issue #116: BOOKKEEPER-1008 Move to Netty4.1

2017-05-03 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/116 @eolivelli I get them too. It may be differences in the default value of ulimit. Probably the simplest thing to do it to try and trace down who is leaking all of the file descriptors. It most

[GitHub] bookkeeper issue #110: BOOKKEEPER-391 Support Kerberos authentication of boo...

2017-04-24 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/110 +1 still looks great. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] bookkeeper issue #110: BOOKKEEPER-391 Support Kerberos authentication of boo...

2017-04-21 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/110 @eolivelli Validation of the client side principal in ZK is done through ACLs. The simplest thing to do in our case is to have a white list of clients (possibly with regular expressions). If

[GitHub] bookkeeper issue #110: BOOKKEEPER-391 Support Kerberos authentication of boo...

2017-04-10 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/110 @eolivelli I am OK with having MD5 in there, but it is horribly insecure and should not be used in production. If we do put it in we should deprecate it very quickly afterwards and ask people

[GitHub] bookkeeper issue #128: BOOKKEEPER-1022 Make BookKeeperAdmin implement AutoCl...

2017-04-10 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/128 +1 (non-binding) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] bookkeeper issue #126: BOOKKEEPER-1018: Allow client to select older V2 prot...

2017-04-05 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/bookkeeper/pull/126 @eolivelli I totally agree we do need some test cases for this. The reason that we use the older API is because of the large amount of object allocation that happens with protocol buffers. I