[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1315 Merging this if no more discussion. CI fails for #6786.9 - Which looks unrelated. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-23 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline I know what you focus on and try to solve the problem. I, actually, cannot accept whether your solution is perfect or not, but I agree the problem you told. Thanks for the explaining

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-22 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul let me try to take a step back and try to explain. Zeppelin is going to be used for various use cases, some of which will involve HDFS - Hive/Spark/Phoenix/Hbase etc, some use

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline It's not clear for me. Do you tell me that users store their Hive, Hbase passwords in HDFS? Or does this support reading `jceks` from HDFS? In your description, this PR is for

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-22 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul this work is currently solving for AD, but is not limited to it. Going forward users may choose to store their Hive, Hbase and other data system passwords. These passwords as

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline Thanks for explaining it. I've researched the `jceks`. AFAIK, it doesn't need hadoop-common dependencies. It's included by java.security by default. Can you remove hadoop-common and

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-21 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul in that case, they will not be able to take advantage of storing AD passwords in an encrypted fashion. That said, there will be no functionality loss. --- If your project is set up

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline Do you mean it's fine if some users don't use Hadoop2 and don't want to use jceks features? Isn't there any side effect at all? --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-17 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1315 @jongyoul It does not matter, because we are just depending upon hadoop 2.6 credential api's. So if the user is on Hadoop 1.x, then he just needs to add Hadoop common - 2.6 jars to the classpath

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-16 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1315 @rconline This PR makes Zeppelin server module include Hadoop-common 2.6. How can I change the version of Hadoop? Or don't we need to change it even we use Hadoop 1.x? --- If your project is

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-16 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1315 Merging this if no more discussion. --- 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

[GitHub] zeppelin issue #1315: [ZEPPELIN-530] Added changes for Credential Provider, ...

2016-08-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1315 Thank you @rconline for taking care of this. LGTM! --- 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