Github user gss2002 closed the pull request at:
https://github.com/apache/zeppelin/pull/2886
---
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 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 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 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 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
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 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 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 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 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 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 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 user gss2002 closed the pull request at:
https://github.com/apache/zeppelin/pull/2851
---
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 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 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 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 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 user gss2002 closed the pull request at:
https://github.com/apache/zeppelin/pull/2849
---
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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
78 matches
Mail list logo