[GitHub] zeppelin issue #1493: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-06 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1493 @vinayshukla here is the pull request for the LdapRealm enhancement we discussed last week at HadoopWorld/Strata --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin pull request #1493: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-06 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1493 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm ### What is this PR for? Provides LdapRealm Functionality similar to what Apache Knox provides. This is critical

[GitHub] zeppelin pull request #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-08 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1513 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-11-07 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1513 @nazgul33 refactoring some code fix coming.. To utilize groups and users --- 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

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-08 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1614 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm Functionality similar to Apache Knox ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm

[GitHub] zeppelin issue #1614: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-11-08 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1614 reopen for clean up --- 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 feature enabled and wishes

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-19 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom I think what happens here is this code fires.. which has nothing to do with the fix here.. in LivySparkInterpreter: return livyHelper.interpretInput(line

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-20 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom / @zjffdu @purechoc there is definitely an additional condition. Not sure if it's because the ConcurrentHashMaps are not being used correctly. But the exception doesn't get caught

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-20 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom just tested against my build by catching the exception and rethrowing.. It definitely solves the issue. } catch (LivyNoSessionException e) { throw e

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-14 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1516 ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos Enabled ### What is this PR for? ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught

[GitHub] zeppelin issue #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1458 Well we will just agree to disagree --- 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 feature

[GitHub] zeppelin issue #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1458 Also input validation is secure java coding best practice regardless.. http://www.oracle.com/technetwork/java/seccodeguide-139067.html#5 --- If your project is set up for it, you can reply

[GitHub] zeppelin pull request #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-14 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1513 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm

[GitHub] zeppelin pull request #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-14 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1513 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1513 Rerun build as error is not related to this patch. --- 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

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-14 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1516 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1614: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-11-12 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1614 No problem Let me know if you see any others or if you need me to adjust this code at all --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-15 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1614 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm Functionality similar to Apache Knox ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-15 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1614 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1614: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-11-20 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1614 Going to close and re-open to re kick the tests. These tests have been flaky I dont think the error is related to the patch... 15:38:55,459 ERROR org.apache.zeppelin.AbstractZeppelinIT

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-20 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1614 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm Functionality similar to Apache Knox ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-20 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1614 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-20 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1614 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm Functionality similar to Apache Knox ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm

[GitHub] zeppelin pull request #1614: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-11-20 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1614 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1514: ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with...

2016-10-31 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1514 @prabhjyotsingh I have only seen this issue with livy.sql as it is the only one that shares the HashMap between livy.spark and livy.sql. livy.sparkr and livy.pyspark do not share session info

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-11-02 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1516 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-11-02 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 Reopen to re-check code and error --- 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 feature

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-11-02 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/1516 ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos Enabled ### What is this PR for? ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-11-02 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 @zjffdu went and confirmed the previous error from the pull 19 days ago was in a set of tests that ran completely fine this time. So I think this patch is good to go --- If your project is set

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-11-02 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 @zjffdu I re-run the tests. This error doesn't look to have anything to do with the code change here. Let me know your thoughts and how we want to proceed Tests run: 9, Failures: 0

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-11 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu I tried to do that initially problem is instantiation of the shared object. I can probably consolidate down to one SessionMap but I think if there is sharing needed between sparkSQL

[GitHub] zeppelin pull request #1512: Zeppelin 1516 1546

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1512 Zeppelin 1516 1546 ### What is this PR for? Livy Interpreter gets a NestedRuntimException when running with Kerberized components. ERROR [2016-10-11 22:44:47,769] ( {pool-2-thread-11

[GitHub] zeppelin pull request #1493: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-13 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1493 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin issue #1493: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1493 @khalidhuseynov I'm going to make the changes and move this under realms. I will also close this pull request and open a new one --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 Also Static HashMaps are not threadsafe with multiple puts in the case of multiple users. Hence why I built out Singletons to keep track of these objects to guarantee only one instance gets

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1516 ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos Enabled ### What is this PR for? ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught with Kerberos

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 @zjffdu and @prabhjyotsingh here is the 404 NestedRuntime Fix --- 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

[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1426 @zjffdu and @prabhjyotsingh the new static Maps can these be converted to ConcurrentHashMaps's to prevent possible future contention with multiple users/threads. https://dzone.com

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu testing the changes again with another build. I see a slight difference between your's and mine. Testing again ill advise shortly --- If your project is set up for it, you can reply

[GitHub] zeppelin pull request #1512: Zeppelin 1516 1546

2016-10-13 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1512 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1513 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm

[GitHub] zeppelin issue #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1513 @khalidhuseynov made the requested changes and updated documentation. Please let me know what you think. Also I will be willing to create a jira and move the other Realms. --- If your project

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu I tried that same code multiple times and it did not work to be completely honest.. I have an email out to a JDK Developer/Debugger for one of the large JDK vendors to find out why

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu The change you made solved the issue. I see difference now. Your quick workaround is good to go. I am going to fix the LivyHelper to handle the 404 with NestedException if that works

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom and @zjffdu is it possible to add a null check also. I've been doing some debugging over the past few days and certain situations can cause nulls to be returned and in theory if a null

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu Also those new maps added with ZEPPELIN-1430 those should be changed to ConcurrentHashMap. Basically any map that is static with gets/puts/removes etc should be ConcurrentHashMap

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu Not a problem. Also I assume we will want to fix SparkR and PySpark also to use ConcurrentHashMaps since they could have multiuser's hitting it. I noticed some of the other interpreters

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu this may be me misunderstanding the code. but if you have 10 users in Zeppelin all accessing PySpark or SparkR there is no possibility of multiple users stepping on the map? So you

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-10 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 Attempted to use the work around (protected static Map<String, Integer> userSessionMap = new HashMap<>(); ) provided by @zjffdu it did not work. So I went with the more invol

[GitHub] zeppelin pull request #1503: NPE LivySparkSQLInterpreter thrown with %livy.s...

2016-10-10 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1503 NPE LivySparkSQLInterpreter thrown with %livy.sql interpreter function ### What is this PR for? The LivySparkSQLInterpreter class does not correctly process the userSessionMap throwing back

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom this fix is good. Did some extensive load testing with it this AM and it solves the session expiration issues. Thanks for the contribution --- If your project is set up for it, you can

[GitHub] zeppelin issue #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1513 @khalidhuseynov and @zjffdu can we look at committing this since tests have passed. If not let me know what else is needed. Thanks --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 @zjffdu and @prabhjyotsingh is this safe to be committed at this point? --- 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

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2017-03-07 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu all set --- 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 feature enabled and wishes so

[GitHub] zeppelin pull request #1503: NPE LivySparkSQLInterpreter thrown with %livy.s...

2017-03-07 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1503 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2017-05-17 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1516 --- 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 feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #2849: ZEPPELIN-3309. Import/Clone doesn't set user in...

2018-03-07 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/2849 ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE ### What is this PR for? During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium and other

[GitHub] zeppelin issue #2849: ZEPPELIN-3309. Import/Clone doesn't set user in Paragr...

2018-03-07 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2849 @zjffdu no problem. yes I will work on a unit test later tonight/tomorrow am. ---

[GitHub] zeppelin pull request #2849: ZEPPELIN-3309. Import/Clone doesn't set user in...

2018-03-08 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/2849 ---

[GitHub] zeppelin issue #2849: ZEPPELIN-3309. Import/Clone doesn't set user in Paragr...

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2849 Closing this pull request for now will re-open with a new one which includes formatting changes and corrections and test cases. ---

[GitHub] zeppelin pull request #2851: ZEPPELIN-3309. Import/Clone doesn't set user in...

2018-03-08 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/2851 ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE What is this PR for? During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium and other

[GitHub] zeppelin issue #2855: ZEPPELIN-3309. Import/Clone user not set in Paragraph ...

2018-03-10 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2855 @zjffdu the only test not passing was the e2e test which I am re-runnin gnow.. Let me know if I need to do anything else. ---

[GitHub] zeppelin pull request #2851: ZEPPELIN-3309. Import/Clone doesn't set user in...

2018-03-08 Thread gss2002
GitHub user gss2002 reopened a pull request: https://github.com/apache/zeppelin/pull/2851 ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE What is this PR for? During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium and other

[GitHub] zeppelin issue #2851: ZEPPELIN-3309. Import/Clone user not set in Paragraph ...

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2851 @zjffdu thanks for the insight. It's setup and running now. ---

[GitHub] zeppelin pull request #2851: ZEPPELIN-3309. Import/Clone user not set in Par...

2018-03-08 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/2851 ---

[GitHub] zeppelin issue #2851: ZEPPELIN-3309. Import/Clone doesn't set user in Paragr...

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2851 @zjffdu any chance you would possibly know why the commit wouldn't be found? [2018-03-08 22:14:11] Can't find build for commit 091130073ee545a4f637d53708ee1f7165bddebc from gss2002 ---

[GitHub] zeppelin pull request #2855: ZEPPELIN-3309. Import/Clone user not set in Par...

2018-03-09 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/2855 ZEPPELIN-3309. Import/Clone user not set in Paragraph causes NPE. What is this PR for? During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium and other functio

[GitHub] zeppelin issue #2855: ZEPPELIN-3309. Import/Clone user not set in Paragraph ...

2018-03-09 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2855 @zjffdu I think this pull request should be good. There looks to be some issues with travis unrelated to the code changes. And I ran the tests a few times not sure whats up with it. ---

[GitHub] zeppelin issue #2855: ZEPPELIN-3309. Import/Clone user not set in Paragraph ...

2018-03-09 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2855 @zjffdu any reason why travis-ci is failing on disk issues? travis_time:end:20c877c0:start=1520617145805371744,finish=1520617145811589622,duration=6217878

[GitHub] zeppelin pull request #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage relog...

2018-04-12 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/2886 ---

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-28 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 https://stackoverflow.com/questions/38555244/how-do-you-set-the-kerberos-ticket-lifetime-from-java https://bugs.openjdk.java.net/browse/JDK-8044500 ---

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-28 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @zjffdu you cannot just update the krb5.conf those are just recommendations on the client side. The KDC both with MIT Krb5 and Active Directory control the max_renewable_lifetime via /var

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-29 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @prabhjyotsingh @zjffdu I made changes to check if security is enabled and if it was logged in via a keytab and than i relogin with checktgt method vs relogining in every time causing excess load

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-29 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @zjffdu I am going to cut the new improved fix based on original feedback. But yes you will have to adjust the KDC to test this as Java does not use ticket_lifetime or renew_lifetime from krb5

[GitHub] zeppelin pull request #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage relog...

2018-03-21 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/2886 ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromKeytab needed What is this PR for? During long runs of Apache Zeppelin using HDFS as the backing configuration and notebook storage. We

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-21 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @zjffdu here is a patch that I think will fix this issue. I will know in 7 days if the issue comes back but has plagued our 4 different environments running Zeppelin over the last few days since

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-22 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @prabhjyotsingh @zjffdu can you help review if you feel this is a valid fix? Thanks again ---

[GitHub] zeppelin issue #2886: ZEPPELIN-3356: Zeppelin FileSystemStorage reloginFromK...

2018-03-23 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/2886 @prabhjyotsingh I just read that same stackoverflow part of me says use checktgtandreloginfronkeytab to be lighter on kdc thoughts? I will dig a bit deeper in am but auto renewal thread