[GitHub] kafka pull request #4310: DOCU: Add authorizer.class.name to the security se...

2017-12-10 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/4310 DOCU: Add authorizer.class.name to the security section in documentation The section _7.4 Authorization and ACLs_ in Kafka documentation describes how to use ACLs. But it doesn't seem to contain

[GitHub] kafka-site pull request #99: Rephrase mailing lists section to avoid incorre...

2017-10-18 Thread scholzj
Github user scholzj commented on a diff in the pull request: https://github.com/apache/kafka-site/pull/99#discussion_r145347106 --- Diff: contact.html --- @@ -11,16 +11,16 @@ - us...@kafka.apache.org

[GitHub] kafka-site pull request #99: Rephrase mailing lists section to avoid incorre...

2017-10-18 Thread scholzj
Github user scholzj commented on a diff in the pull request: https://github.com/apache/kafka-site/pull/99#discussion_r145345710 --- Diff: contact.html --- @@ -11,16 +11,16 @@ - us...@kafka.apache.org

[GitHub] kafka-site issue #81: MINOR: Fix typo in the gradle command for code style c...

2017-10-18 Thread scholzj
Github user scholzj commented on the issue: https://github.com/apache/kafka-site/pull/81 Can someone have a look at this? It is just a simple typo, so it should be very easy to review and merge. ---

[GitHub] kafka-site pull request #99: Rephrase mailing lists section to avoid incorre...

2017-10-18 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka-site/pull/99 Rephrase mailing lists section to avoid incorrect subscribe emails The user@ and dev@ mailing lists seem to receive lot of emails from users who are not subscribed but are trying to subscribe. I

[GitHub] kafka pull request #4012: KAFKA-6008: Sanitize the Kafka Connect workerId be...

2017-10-04 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/4012 KAFKA-6008: Sanitize the Kafka Connect workerId before passing it to AppInfoParser You can merge this pull request into a Git repository by running: $ git pull https://github.com/scholzj

[GitHub] kafka pull request #3920: KAFKA-5940: Throw exception if the JSON file is in...

2017-09-29 Thread scholzj
Github user scholzj closed the pull request at: https://github.com/apache/kafka/pull/3920 ---

[GitHub] kafka pull request #3955: KAFKA-5969: Use correct error message when the JSO...

2017-09-25 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3955 KAFKA-5969: Use correct error message when the JSON file is invalid When invalid JSON file is passed to the bin/kafka-preferred-replica-election.sh / PreferredReplicaLeaderElectionCommand tool

[GitHub] kafka pull request #3920: KAFKA5940: Throw exception if the JSON file is inv...

2017-09-20 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3920 KAFKA5940: Throw exception if the JSON file is invalid Throw an exception when the JSON file with topics/partitions and offsets is incorrect (= when Json.parseFull(...) returns None). You can

[GitHub] kafka-site pull request #81: MINOR: Fix typo in the gradle command for code ...

2017-09-20 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka-site/pull/81 MINOR: Fix typo in the gradle command for code style check Fix typo in the gradle command for code style check: chec**k**styleMain instead of chec**h**styleMain You can merge this pull request

[GitHub] kafka pull request #3908: KAFKA-5933: Move timeout and validate_only protoco...

2017-09-19 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3908 KAFKA-5933: Move timeout and validate_only protocol fields into CommonFields class Most of the fields which are shared by multiple protocol messages (requests ad responses) are in the CommonFields

[GitHub] kafka pull request #3898: KAFKA-4320: Log cleaner is enabled by default

2017-09-19 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3898 KAFKA-4320: Log cleaner is enabled by default The log cleaner is enabled by default since 0.9.0.1. But the 0.9.0 docs are not updated. This PR updates the docs to say that it is enabled by default

[GitHub] kafka pull request #3900: KAFKA-4320: Clarify the log compaction config prop...

2017-09-19 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3900 KAFKA-4320: Clarify the log compaction config property per-topic and per-broker The per-topic config property is different from the per-broker. This PR makes it a bit more clear. You can merge

[GitHub] kafka pull request #3899: KAFKA-4320: Log cleaner is enabled by default

2017-09-19 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3899 KAFKA-4320: Log cleaner is enabled by default The log cleaner is enabled by default since 0.9.0.1. But the 0.10.0 docs are not updated. This PR updates the docs to say that it is enabled by default

[GitHub] kafka pull request #3883: KAFKA-5918: Fix minor typos and errors in the Kafk...

2017-09-17 Thread scholzj
GitHub user scholzj opened a pull request: https://github.com/apache/kafka/pull/3883 KAFKA-5918: Fix minor typos and errors in the Kafka Streams turotial I found several minor issues with the Kafka Streams tutorial: * Some typos * "As shown above, it illus