Andor Molnár

icon indicating a website link http://zookeeper.apache.org/ icon indicating an email [email protected]

icon company @cloudera icon location Austin, TX icon location Individual contributor at Cloudera. Apache ZooKeeper PMC.

Results 120 comments of Andor Molnár

ZOOKEEPER-2590:exists() should check read ACL permission

I changed my mind, implementing the parent ACL check doesn't make much sense. The same 'scan' issue can be achieved with get too: ``` /a - protected /a/b - protected...

ZOOKEEPER-2590:exists() should check read ACL permission

@eolivelli @ztzg @phunt Unfortunately @maoling is not active on this patch, I suggest to fork his patch and create a new PR to move forward with the fix.

ZOOKEEPER-2590:exists() should check read ACL permission

> I agree. But I don't have cycles No worries, I'll do it.

ZOOKEEPER-2590:exists() should check read ACL permission

Closing this PR in favor of #2093

ZOOKEEPER-4529 : Upgrade netty to 4.1.75.Final

outdated.

ZOOKEEPER-4787:Failed to establish connection between zookeeper

@Softrock Looks fine, but please open a PR to master branch only.

ZOOKEEPER-4708: recreateSocketAddresses before comparing addresses

@showuon You have checkstyle validation error ``` Error: /home/runner/work/zookeeper/zookeeper/zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/LeaderBeanTest.java:43:1: Extra separation in import group before 'org.apache.jute.OutputArchive' [ImportOrder] ```

ZOOKEEPER-4760. Added options in get and set commands.

> > Please create a Jira for this and describe the feature mentioning which version branch would you like it to be added. I think this is suitable for `master`...

ZOOKEEPER-3000: Use error-prone compiler

Also would you please elaborate a little bit more on reasoning in Jira's description for the records?

ZOOKEEPER-3000: Use error-prone compiler

@maoling [hbase](https://github.com/apache/hbase/blob/f63a7ff635fa02a8a97d9a36505732f647d2e7ff/hbase-build-configuration/pom.xml) do use errorprone. At least in some modules.