[GitHub] zeppelin pull request #2922: [ZEPPELIN-3403] Zeppelin documentation not poin...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2922


---

[GitHub] zeppelin pull request #2909: [ZEPPELIN-3384] Reduce the number of calls that...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2909


---

[GitHub] zeppelin pull request #2915: ZEPPELIN-3391 Incorrect status shown for '%livy...

[r-kamath]

Github user r-kamath closed the pull request at:

https://github.com/apache/zeppelin/pull/2915


---

[GitHub] zeppelin issue #2915: ZEPPELIN-3391 Incorrect status shown for '%livy2.conf'...

[r-kamath]

Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/2915
  
right fix is in #2926


---

[GitHub] zeppelin issue #2926: ZEPPELIN-3391. Incorrect status shown for '%livy2.conf...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2926
  
@felixcheung @r-kamath Could you help review it ? Thanks


---

[GitHub] zeppelin pull request #2926: ZEPPELIN-3391. Incorrect status shown for '%liv...

[zjffdu]

GitHub user zjffdu opened a pull request:

https://github.com/apache/zeppelin/pull/2926

ZEPPELIN-3391. Incorrect status shown for '%livy2.conf' and %spark2.conf' 
interpreters

### What is this PR for?
The root cause of this bug is that the final status of paragraph is not 
determined by interpreter, but also by its scheduler. Here's the bug is in 
FIFOScheduler. I just fix it as a quick solution in this PR. For a long term 
solution, I think we may need to do code refactoring in the scheduler 
component. 

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3391

### How should this be tested?
* Manually tested

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3391

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2926.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2926


commit 09441ad1356fbab50d7ad25057653d0388ad6c41
Author: Jeff Zhang 
Date:   2018-04-11T14:53:48Z

ZEPPELIN-3391. Incorrect status shown for '%livy2.conf' and %spark2.conf' 
interpreters




---

[GitHub] zeppelin pull request #2925: ZEPPELIN-3404. Fail to run cronjob when user do...

[zjffdu]

GitHub user zjffdu opened a pull request:

https://github.com/apache/zeppelin/pull/2925

ZEPPELIN-3404. Fail to run cronjob when user doesn't run it manually before 
cronjob

### What is this PR for?

This bug is introduced by #2914, this PR will set authenticationInfo using 
its user as we will store user into note.json.  


### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3404

### How should this be tested?
* Manually tested

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3404

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2925.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2925


commit aae17c11ada4eadc48380183386c04d74e1a6603
Author: Jeff Zhang 
Date:   2018-04-13T02:31:45Z

ZEPPELIN-3404. Fail to run cronjob when user doesn't run it manually before 
cronjob




---

[GitHub] zeppelin issue #2914: [ZEPPELIN-3350] Don't allow set cronExecutionUser

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2914
  
Thanks @weand I have created ZEPPELIN-3404 for it


---

[GitHub] zeppelin issue #2914: [ZEPPELIN-3350] Don't allow set cronExecutionUser

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2914
  
Thanks @weand I have created ZEPPELIN-3404 for it


---

[jira] [Created] (ZEPPELIN-3404) Fail to run cronjob is user don't run it manually before cronjob

[Jeff Zhang (JIRA)]

Jeff Zhang created ZEPPELIN-3404:


 Summary: Fail to run cronjob is user don't run it manually before 
cronjob
 Key: ZEPPELIN-3404
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3404
 Project: Zeppelin
  Issue Type: Bug
Reporter: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] zeppelin issue #2923: ZEPPELIN-3312 Add option to convert username to lower ...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2923
  
LGTM @prabhjyotsingh could you help take a look at the npm error in travis 
? Some other contributors also hit this same in other PR recently. 


---

[GitHub] zeppelin pull request #:

[bern80]

Github user bern80 commented on the pull request:


https://github.com/apache/zeppelin/commit/f4a798b6230b4f2389ad7ea9b41818b15d9ab5a7#commitcomment-28558190
  
How about ui-grid exporter dependencies?
E.g. Currently Table export to excel is not working due to missing 
dependencies:

[(https://github.com/angular-ui/ui-grid/blob/18c5ffb8807122e6045643c73138b9d6c83b5f9e/misc/tutorial/206_exporting_data.ngdoc]



---

[GitHub] zeppelin issue #2923: ZEPPELIN-3312 Add option to convert username to lower ...

[VipinRathor]

Github user VipinRathor commented on the issue:

https://github.com/apache/zeppelin/pull/2923
  
All Travis CI builds are failing for some npm error. Looks unrelated but 
I'm not sure. I need some help in getting this through.
CC: @prabhjyotsingh @r-kamath 


---

[GitHub] zeppelin pull request #2923: ZEPPELIN-3312 Add option to convert username to...

[VipinRathor]

GitHub user VipinRathor reopened a pull request:

https://github.com/apache/zeppelin/pull/2923

ZEPPELIN-3312 Add option to convert username to lower case

### What is this PR for?
This PR introduces a new configuration property to convert username to 
lower case. This is useful when the users (from external sources like AD/LDAP) 
are coming in with mixed-case names and Hadoop services (like Hive) can't 
authorize them correctly because Hadoop services recognize users only in lower 
case (like Linux). 

Adding a new config option "zeppelin.username.force.lowercase" to handle 
such scenarios.

Behavior without this PR:
Access is denied to CaMel case user while running a Hive paragraph

Behavior with this PR:
User is allowed to run query when proposed configuration set to true.
By default, keeping zeppelin.username.force.lowercase=false to retain the 
current behavior.

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Unit test

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3312

### How should this be tested?
* Travis CI should pass
* Manual steps to test:
1. Configure Zeppelin with Active Directory authentication
2. Login to Zeppelin as a CaMel case user
3. Try to run a simple JDBC note with a Hive query (like a select * query). 
This would fail with "user [CaMel] does not have proper privileges to [USE] 
operation" error message.
4. Now set zeppelin.username.force.lowercase=true in custom 
zeppelin-site.xml configuration.
5. Once again, login as CaMel case user. This time the same Hive query 
would run as expected. Because the username is now passed in lower case.
6. Also notice that after successful login, the login username (in the 
top-right corner) will be in lower case too.

### Screenshots (if appropriate)
* Login as CaMel case user:
https://user-images.githubusercontent.com/15668387/38672744-faf00f8e-3e03-11e8-86b2-cc5981d380d2.png;>
* Notice the converted username post login:
https://user-images.githubusercontent.com/15668387/38672777-108c7b66-3e04-11e8-8c97-467b4b73fe3d.png;>

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/VipinRathor/zeppelin ZEPPELIN-3312

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2923.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2923


commit 1a8b2d94eb449d7fac10f852e3af81e44d3a5e76
Author: Vipin Rathor 
Date:   2018-04-12T10:23:04Z

Add support to force username case conversion

commit 38337297e6e7c80f2a776f8fbdeabe5b5fe06f59
Author: VipinRathor 
Date:   2018-04-12T10:52:41Z

Merge branch 'master' into ZEPPELIN-3312




---

[GitHub] zeppelin pull request #2923: ZEPPELIN-3312 Add option to convert username to...

[VipinRathor]

Github user VipinRathor closed the pull request at:

https://github.com/apache/zeppelin/pull/2923


---

[GitHub] zeppelin issue #2914: [ZEPPELIN-3350] Don't allow set cronExecutionUser

[weand]

Github user weand commented on the issue:

https://github.com/apache/zeppelin/pull/2914
  
not directly when logged out. instead the NPE happens when Zeppelin gets 
restartet and next cron runs before the note was run manually, which is a 
frequent use case IMHO.
> ERROR [2018-04-12 21:11:00,024] ({DefaultQuartzScheduler_Worker-1} 
QuartzScheduler.java[schedulerError]:2425) - Job (note.2DBFZ9GGT threw an 
exception.
org.quartz.SchedulerException: Job threw an unhandled exception. [See 
nested exception: java.lang.NullPointerException: while trying to invoke the 
method org.apache.zeppelin.user.AuthenticationInfo.getUser() of a null object 
loaded from local variable 'authenticationInfo']
at org.quartz.core.JobRunShell.run(JobRunShell.java:213)
at 
org.quartz.simpl.SimpleThreadPool$WorkerThread.run(SimpleThreadPool.java:573)
Caused by: java.lang.NullPointerException: while trying to invoke the 
method org.apache.zeppelin.user.AuthenticationInfo.getUser() of a null object 
loaded from local variable 'authenticationInfo'
at 
org.apache.zeppelin.notebook.Paragraph.setAuthenticationInfo(Paragraph.java:209)
at org.apache.zeppelin.notebook.Paragraph.execute(Paragraph.java:353)
at org.apache.zeppelin.notebook.Note.run(Note.java:678)
at org.apache.zeppelin.notebook.Note.runAll(Note.java:659)
at org.apache.zeppelin.notebook.Note.runAll(Note.java:648)
at 
org.apache.zeppelin.notebook.Notebook$CronJob.execute(Notebook.java:936)
at org.quartz.core.JobRunShell.run(JobRunShell.java:202)
... 1 more


---

[GitHub] zeppelin issue #2916: ZEPPELIN-3395 Fix: Spark interpreter does not run as i...

[iijima-satoshi]

Github user iijima-satoshi commented on the issue:

https://github.com/apache/zeppelin/pull/2916
  
Tests for interpreters except spark one were droped in case of 
ZEPPELIN_IMPERSONATE_SPARK_PROXY_USER = true.
I fixed and manually tested.



---

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

[gss2002]

Github user gss2002 closed the pull request at:

https://github.com/apache/zeppelin/pull/2886


---

[GitHub] zeppelin pull request #2921: Zeppelin npm error: getaddrinfo ENOTFOUND

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2921


---

[GitHub] zeppelin issue #2923: ZEPPELIN-3312 Add option to convert username to lower ...

[Tagar]

Github user Tagar commented on the issue:

https://github.com/apache/zeppelin/pull/2923
  
Great idea.. I had a similar jira 
https://issues.apache.org/jira/browse/ZEPPELIN-2886 
Thanks!


---

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

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2886
  
@gss2002 I just found the root cause is that 
UserGroupInformation.loginUserFromKeytab called multiple times and created PR 
#2924 to fix it,  I have verified it, could you help verified it if you have 
time ?


---

[GitHub] zeppelin pull request #2924: [ZEPPELIN-3356] Kerberos ticket still expire af...

[zjffdu]

GitHub user zjffdu opened a pull request:

https://github.com/apache/zeppelin/pull/2924

[ZEPPELIN-3356] Kerberos ticket still expire after 7 days

### What is this PR for?

The root cause is that we may do UserGroupInformation.loginUserFromKeytab 
multiple times if we use hdfs for several places, e.g. for notebook repo and 
zeppelin config. This PR fix the bug by only doing 
UserGroupInformation.loginUserFromKeytab one time.


### What type of PR is it?
[Bug Fix ]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3356

### How should this be tested?
Manually tested
* Add the following properties to /etc/krb5.conf
   renew_lifetime = 10m
  ticket_lifetime = 5m
* Add the following properties to /var/kerberos/krb5kdc/kdc.conf
  max_renewable_life = 10m
  max_life = 5m

Without this PR, will hit the ticket expire after 10 minutes. With this PR, 
the ticket is still valid after 10 minutes 


### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3356

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2924.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2924






---

[GitHub] zeppelin issue #2921: Zeppelin npm error: getaddrinfo ENOTFOUND

[r-kamath]

Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/2921
  
will merge this if there are no more discussions


---

[GitHub] zeppelin issue #2923: ZEPPELIN-3312 Add option to convert username to lower ...

[VipinRathor]

Github user VipinRathor commented on the issue:

https://github.com/apache/zeppelin/pull/2923
  
@prabhjyotsingh @felixcheung @zjffdu Can you please help review? Thanks.


---

[GitHub] zeppelin pull request #2923: ZEPPELIN-3312 Add option to convert username to...

[VipinRathor]

GitHub user VipinRathor opened a pull request:

https://github.com/apache/zeppelin/pull/2923

ZEPPELIN-3312 Add option to convert username to lower case

### What is this PR for?
This PR introduces a new configuration property to convert username to 
lower case. This is useful when the users (from external sources like AD/LDAP) 
are coming in with mixed-case names and Hadoop services (like Hive) can't 
authorize them correctly because Hadoop services recognize users only in lower 
case (like Linux). 

Adding a new config option "zeppelin.username.force.lowercase" to handle 
such scenarios.

Behavior without this PR:
Access is denied to CaMel case user while running a Hive paragraph

Behavior with this PR:
User is allowed to run query when proposed configuration set to true.
By default, keeping zeppelin.username.force.lowercase=false to retain the 
current behavior.

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Unit test

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3312

### How should this be tested?
* Travis CI should pass
* Manual steps to test:
1. Configure Zeppelin with Active Directory authentication
2. Login to Zeppelin as a CaMel case user
3. Try to run a simple JDBC note with a Hive query (like a select * query). 
This would fail with "user [CaMel] does not have proper privileges to [USE] 
operation" error message.
4. Now set zeppelin.username.force.lowercase=true in custom 
zeppelin-site.xml configuration.
5. Once again, login as CaMel case user. This time the same Hive query 
would run as expected. Because the username is now passed in lower case.
6. Also notice that after successful login, the login username (in the 
top-right corner) will be in lower case too.

### Screenshots (if appropriate)
* Login as CaMel case user:
https://user-images.githubusercontent.com/15668387/38672744-faf00f8e-3e03-11e8-86b2-cc5981d380d2.png;>
* Notice the converted username post login:
https://user-images.githubusercontent.com/15668387/38672777-108c7b66-3e04-11e8-8c97-467b4b73fe3d.png;>

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/VipinRathor/zeppelin ZEPPELIN-3312

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2923.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2923


commit 1a8b2d94eb449d7fac10f852e3af81e44d3a5e76
Author: Vipin Rathor 
Date:   2018-04-12T10:23:04Z

Add support to force username case conversion




---

[GitHub] zeppelin issue #2922: [ZEPPELIN-3403] Zeppelin documentation not pointing to...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2922
  
Will merge this to master and branch-0.8 if no more discussion.


---

[GitHub] zeppelin issue #2922: [ZEPPELIN-3403] Zeppelin documentation not pointing to...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2922
  
@zjffdu  Can you help review this. 


---

[GitHub] zeppelin pull request #2922: [ZEPPELIN-3403] Zeppelin documentation not poin...

[prabhjyotsingh]

GitHub user prabhjyotsingh opened a pull request:

https://github.com/apache/zeppelin/pull/2922

[ZEPPELIN-3403] Zeppelin documentation not pointing to correct url

### What is this PR for?
Zeppelin documentation not pointing to correct URL. 

Zeppelin documentation is redirecting to 
http://zeppelin.apache.org/docs//index.html and this URL doesn't exist.

The available doc for 0.8.0 is 
http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/index.html

### What type of PR is it?
[Bug Fix]

### What is the Jira issue?
* [ZEPPELIN-3403](https://issues.apache.org/jira/browse/ZEPPELIN-3403)


### Questions:
* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-3403

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2922.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2922


commit beb2277dc1d51691ddd88d86f62c4896c612e3b9
Author: Prabhjyot Singh 
Date:   2018-04-12T07:34:16Z

ZEPPELIN-3403: Zeppelin documentation not pointing to correct url

Change-Id: Id34af509e63bf633e86e2d09173e5326baaefcb4




---

[jira] [Created] (ZEPPELIN-3403) Zeppelin documentation not pointing to correct url

[Supreeth Sharma (JIRA)]

Supreeth Sharma created ZEPPELIN-3403:
-

 Summary: Zeppelin documentation not pointing to correct url
 Key: ZEPPELIN-3403
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3403
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-server
Affects Versions: 0.8.0
Reporter: Supreeth Sharma
 Fix For: 0.8.0


Zeppelin documentation not pointing to correct url. 

Zeppelin documentation is redirecting to 
[http://zeppelin.apache.org/docs//index.html] and this url doesn't exist.

The available doc for 0.8.0 is 
http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/index.html

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[GitHub] zeppelin issue #2918: [ZEPPELIN-3399] Tests failing with no changes - ShellS...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2918
  
LGTM @ReedOei Could you trigger the build again ? I don't see the travis 
succeeded


---

[jira] [Created] (ZEPPELIN-3402) ParagraphActionsIT fails on travis

[Jeff Zhang (JIRA)]

Jeff Zhang created ZEPPELIN-3402:


 Summary: ParagraphActionsIT fails on travis
 Key: ZEPPELIN-3402
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3402
 Project: Zeppelin
  Issue Type: Bug
Reporter: Jeff Zhang


[~prabhjyotsi...@apache.com] [~renjithgk] Could you any of you take a look at 
this ? Thanks



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)