Colin Patrick McCabe

Interesting. Were you using SELinux?

Thanks for this fix! One problem I see is that this will allow the deletion to partly go through, until we run out of records. It would be better for...

We need a boolean to indicate that there is more data that we were unable to send due to length restrictions

Can you give a code example?

The build failed: ``` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.6.1:testCompile (default-testCompile) on project kafka-rest: Compilation failure [ERROR] /home/jenkins/workspace/entinc-pr_kafka-rest_PR-435-42EK4ZKFX7WUDVKYQNLWCJBUA3J5IBRM4CNHID3FE32PTTTFMBRQ/kafka-rest/src/test/java/io/confluent/kafkarest/TestUtils.java:[263,34] cannot find symbol [ERROR] symbol: method createConsumerProperties(java.lang.String,java.lang.String,java.lang.String,int) [ERROR] location: class kafka.utils.TestUtils [ERROR]...

This PR shouldn't need to depend on ZooKeeper. We should be able to use the public AdminClient. What's the security model here? How do we decide who can and cannot...

> Zookeeper needs to get broker information. I mean security protocol in EndPoint class. Why is this information needed for REST proxy? If it is needed, then we should at...

> Interesting. This is an issue because the ZK cluster provisioned by the system tests is the latest version and not a version that supports the old ZK clients? Unfortunately,...

Yes, I think you're right. It should be AK 2.4 that's the cutoff. not AK 3.5. Will fix.

> @cmccabe are you planning to run Confluent's system test job on this branch/pr? I ran a few variants locally using ducker and it seems good.