[GitHub] storm issue #2062: [STORM-2470] kafkaspout should support kerberos

2017-04-12 Thread liu-zhaokun
Github user liu-zhaokun commented on the issue: https://github.com/apache/storm/pull/2062 Users should provide the path to jaas.conf which includes a section named "KafkaClient" in storm.yaml or . --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] storm issue #2062: [STORM-2470] kafkaspout should support kerberos

2017-04-12 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/2062 @liu-zhaokun with the latest kafka consumer APIs one can pass a keytab and principal via consumer or producer properties. Even before that they can pass the jaas config via JVM param and set securit

[GitHub] storm issue #2062: [STORM-2470] kafkaspout should support kerberos

2017-04-12 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/2062 @liu-zhaokun More details are documented here http://kafka.apache.org/documentation.html#security_sasl . We don't need these changes on storm-kafka-client side --- If your project is set up for it

[GitHub] storm issue #2062: [STORM-2470] kafkaspout should support kerberos

2017-04-12 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/2062 @liu-zhaokun KafkaSpoutConfig offers this method to add security-related configs https://github.com/apache/storm/blob/master/external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spou

[GitHub] storm issue #2062: [STORM-2470] kafkaspout should support kerberos

2017-04-12 Thread liu-zhaokun
Github user liu-zhaokun commented on the issue: https://github.com/apache/storm/pull/2062 Hello, @harshach Yes,you are right.But I found there isn't 'sasl.jaas.config' in kafka 0.9.x, so I will create a new PR for storm 1.0.x-branch. --- If your project is set up for it, you can