[GitHub] zeppelin pull request #1353: ZEPPELIN-1326: make profile to select dependenc...

2016-08-22 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

ZEPPELIN-1326: make profile to select dependency of hadoop-common for JDBC 
interpreter

### What is this PR for?
java.lang.ClassNotFoundException: 
org.apache.hadoop.security.UserGroupInformation$AuthenticationMethod error on 
using jdbc interpreter after #1205 

```
java.lang.ClassNotFoundException: 
org.apache.hadoop.security.UserGroupInformation$AuthenticationMethod
at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
at 
org.apache.zeppelin.jdbc.security.JDBCSecurityImpl.getAuthtype(JDBCSecurityImpl.java:71)
at 
org.apache.zeppelin.jdbc.JDBCInterpreter.getConnection(JDBCInterpreter.java:217)
at 
org.apache.zeppelin.jdbc.JDBCInterpreter.getStatement(JDBCInterpreter.java:275)
at 
org.apache.zeppelin.jdbc.JDBCInterpreter.executeSql(JDBCInterpreter.java:346)
at 
org.apache.zeppelin.jdbc.JDBCInterpreter.interpret(JDBCInterpreter.java:452)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:94)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServer$InterpretJob.jobRun(RemoteInterpreterServer.java:383)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at 
org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
```


Workaround is adding "org.apache.hadoop:hadoop-common:2.7.2" in the 
Dependencies section of interpreter setting on GUI. But jdbc interpreter 
supposed to work without asking user hadoop-common dependency.

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


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

### How should this be tested?
On running this (JDBC) interpreter, the above stack trace should not come.

### Screenshots (if appropriate)

### 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-1326

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

https://github.com/apache/zeppelin/pull/1353.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 #1353


commit f061da03aed48609d07cab2eed199369adb26267
Author: Prabhjyot Singh 
Date:   2016-08-23T05:30:29Z

ZEPPELIN-1326: make profile to select dependency for hadoop-common




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #989: Added ipfsnotebookrepo [ Zeppelin-683 ]

2016-08-22 Thread onkarshedge
Github user onkarshedge closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #989: Added ipfsnotebookrepo [ Zeppelin-683 ]

2016-08-22 Thread onkarshedge
GitHub user onkarshedge reopened a pull request:

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

 Added ipfsnotebookrepo [ Zeppelin-683 ]

### What is this PR for?
IPFSNoteBookRepo IPFS storage and sharing zeppelin notes.


### What type of PR is it?
 Feature 

### Todos
* [x] checkpoint revisions, get particular revision
* [x] List of revision history
* [x] remove note and also a particular revision
* [x] support to import the Note when hash is given as input
* [x] Documentation 

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

### How should this be tested?
You need the [ipfs 
jar](https://github.com/ipfs/java-ipfs-api/blob/master/pom.xml) in the local 
.m2 repository. `mvn install:install-file` hers is complete 
[command](https://maven.apache.org/guides/mini/guide-3rd-party-jars-local.html)
Also please refer the `storage document` for more help.

### Screenshots (if appropriate)
[Successful import](https://gifyu.com/image/52t)
[Failure import](https://gifyu.com/image/52Y)

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



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

$ git pull https://github.com/onkarshedge/incubator-zeppelin ipfs

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

https://github.com/apache/zeppelin/pull/989.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 #989


commit 61a070449f5d75d53199a3bb41604a48fd175aed
Author: onkarshedge 
Date:   2016-06-10T07:55:59Z

 Added ipfsnotebookrepo with Versioning and import feature
 backend and frontend

commit 8a6ff363d89571eb8fb25d2af86e5a7080061072
Author: onkarshedge 
Date:   2016-08-17T14:44:47Z

added import status as ng-toast

commit 06040835d6f97a4d6bbce7a6ee274d118aed8ca4
Author: onkarshedge 
Date:   2016-08-18T06:12:42Z

Added documentation

commit b09876d70abf99a448e791363e6363d3cf7c3ac6
Author: onkarshedge 
Date:   2016-08-22T15:48:54Z

refactored code to remove any dependencies and use httpclient




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
Sure, In this PR I was only thinking about the otherwise case i.e. in the 
environment where hadoop dependencies where not present, and hence start 
interpreter as end-web-user. 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1338: Zeppelin 1307 - Implement notebook revision in ...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-22 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1352
  
Got it, makes perfect sense! 

Do you think we should add a comment in there, explaining that 
compatibility is the reason for such decoding? I feel like this might help 
future contributors to see how to deal with such compatibility issue.

Other then that - it looks great to me, thanks again for fixing! It shall 
be ready to merge to master right after that, if there is no further 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1208: [gh-pages][ZEPPELIN-1216] Add a matrix table fo...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1008: [ZEPPELIN-997] Export data to csv

2016-08-22 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1008
  
@qinzl1 It might be independent from the download button.
Queries are usually limited to 1000 rows (it is set in the interpreter's 
setting).
Can you check that?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1008: [ZEPPELIN-997] Export data to csv

2016-08-22 Thread qinzl1
Github user qinzl1 commented on the issue:

https://github.com/apache/zeppelin/pull/1008
  
if i can export more than  1000 rows


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1338
  
tested and works well now! 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1338
  
@khalidhuseynov fixed the issue, thank for testing


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1352: [ZEPPELIN-1327] Fix bug in z.show for Python interpret...

2016-08-22 Thread bustios
Github user bustios commented on the issue:

https://github.com/apache/zeppelin/pull/1352
  
Yes @bzz, I just deleted `if self.py3:` but not `img_str = 
img_str.decode('ascii')` (it is in line 173), which means it will be executed 
for both Python 2 and 3. Decoding bytes to string it necessary for Python 3 
because that version makes distinction between String and Bytes, which does not 
occur in Python 2.

Maybe, `img_str = img_str.decode('ascii')` should be in a separate line of 
code to be clear.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-08-22 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1290
  
@mwkang Can you rebase please?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@astroshim Oh that was my bad. Please ignore my above comment. I only build 
`zeppelin-web` not backend side.
Tested again and it works well as expected. Let's wait the other reviewers 
:)


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-08-22 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
@mfelgamal Thank you! Please let me look into CI and get back to you


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1242: [ZEPPELIN-960] When there is no interpreter, paragraph...

2016-08-22 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1242
  
Thank you for your patience!
Looks good to me, except minor style issue above?

As soon as it is addressed - I think we should be good to merge it to 
master.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1358) Displaying Pandas DataFrame index in table using z.show()

2016-08-22 Thread Paul Bustios (JIRA)
Paul Bustios created ZEPPELIN-1358:
--

 Summary: Displaying Pandas DataFrame index in table using z.show()
 Key: ZEPPELIN-1358
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1358
 Project: Zeppelin
  Issue Type: Improvement
  Components: python-interpreter
Affects Versions: 0.7.0
Reporter: Paul Bustios
Assignee: Paul Bustios
Priority: Minor
 Fix For: 0.7.0


Current implementation of z.show() for python interpreter only displays the 
column names and data of a Pandas DataFrame, being also necessary to display 
the index.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (ZEPPELIN-1357) Category Text lables wrapping not done properly in charts

2016-08-22 Thread nirav patel (JIRA)
nirav patel created ZEPPELIN-1357:
-

 Summary: Category Text lables wrapping not done properly in charts
 Key: ZEPPELIN-1357
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1357
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end, GUI
Affects Versions: 0.6.1
Reporter: nirav patel
 Attachments: Screen Shot 2016-08-22 at 3.35.09 PM.png

I see overlap in text when texts are more then around 15 characters. May be 
spacing in not enough. It should be wrapped or spaced out evenly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin pull request #1352: Fix bug in z.show for Python interpreter

2016-08-22 Thread bustios
GitHub user bustios opened a pull request:

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

Fix bug in z.show for Python interpreter

### What is this PR for?
Currently, height parameter for z.show implementation to display PNG images 
in Python interpreter is not working. This PR fix that bug.

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

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

### How should this be tested?
```python
import matplotlib.pyplot as plt

x = [1,2,3,4,5]
y = [6,7,8,9,0]

plt.plot(x, y, marker="o")
z.show(plt, height="200px")
plt.close()
```

### 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/bustios/zeppelin ZEPPELIN-1327

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

https://github.com/apache/zeppelin/pull/1352.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 #1352


commit 1a78a37d0f9dc2eb92081eb89945863950bb397d
Author: Paul Bustios 
Date:   2016-08-22T21:19:29Z

Fix bug in z.show for Python interpreter




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-22 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
for what it's worth (probably not much) this passed travis CI on my branch- 

[https://travis-ci.org/rawkintrevo/incubator-zeppelin/builds/154183905](https://travis-ci.org/rawkintrevo/incubator-zeppelin/builds/154183905)

Failing on profile 1, onClick in webserver hits time out (unrelated).

[https://travis-ci.org/apache/zeppelin/jobs/154156408#L16359](https://travis-ci.org/apache/zeppelin/jobs/154156408#L16359)



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-22 Thread Peilin-Yang
GitHub user Peilin-Yang opened a pull request:

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

[ZEPPELIN/1356] The graph legend truncates at the nearest period (.) in its 
grouping

### What is this PR for?
Fix the issue: in line graph if user uses the numbers that contains 
period(.), e.g. 3.14 in the groups the legend will only show 3 instead of 3.14.


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

### Todos

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

### How should this be tested?
Localhost test with screenshot

### Screenshots (if appropriate)

https://drive.google.com/a/twitter.com/file/d/0B9QN74Rceec8NG10UEt0R3VUWTg/view

### 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/Peilin-Yang/zeppelin ypeilin/ZEPPELIN-1356

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

https://github.com/apache/zeppelin/pull/1351.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 #1351


commit 00527a5a0c35c0b292348a6b90110de22d8106c5
Author: Peilin Yang 
Date:   2016-08-22T19:40:24Z

fix ZEPPELIN/1356




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1356) The graph legend truncates at the nearest period (.) in its grouping

2016-08-22 Thread Peilin Yang (JIRA)
Peilin Yang created ZEPPELIN-1356:
-

 Summary: The graph legend truncates at the nearest period (.) in 
its grouping
 Key: ZEPPELIN-1356
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1356
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end, GUI, zeppelin-server
Affects Versions: 0.7.0
Reporter: Peilin Yang
Priority: Minor
 Fix For: 0.7.0


To reproduce:
Use the following query
%sh echo -e "%table name\tgroup\tvalue\n1\t2.3.4.5\t10"
After running, select the line graph. Use name as keys, group as groups, and 
value as values.
Expected
Graph legend in top left and overlay displays 2.3.4.5 as a group.
Actual
Graph legend only shows 2 as a group (truncating off the rest of the number).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (ZEPPELIN-1355) Shiro authentication for LDAP SSO

2016-08-22 Thread Bhanuprakash Prathap (JIRA)
Bhanuprakash Prathap created ZEPPELIN-1355:
--

 Summary: Shiro authentication for LDAP SSO
 Key: ZEPPELIN-1355
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1355
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.6.1
 Environment: DEV
Reporter: Bhanuprakash Prathap


We know that Zeppelin has introduced shiro authentication with 0.6.0. Can this 
be used/enhanced to provide SSO/LDAP authentication with our organization LDAP 
Team? We are trying to achieve a common authentication methodology for members 
across the organization.

Any information/help to integrate Zeppelin with LDAP/SSO authentication would 
be great.

Note: Currently our ORG SSO Team default servers for providing SSO is WAS with 
IHS(IBM HTTP Server).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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 cases will also need 
support for non-HDFS such as Postgres/Mysql etc. 

**Problem** - all of these end systems may require users to store 
passwords. Currently in zeppelin there are two locations for storing these 
passwords - 1. shiro.ini for AD passwords and 2. interpreter.json for the rest 
of the data systems. These passwords are stored in clear text as of now. 

**Solution** - Encrypt the password and store in a file that can be read 
only at runtime, by zeppelin process to connect successfully. Question is 
where? Either on the zeppelin host system or hdfs where big-data users are akin 
to storing passwords. JCEKS is a java supported concept and has worked well for 
most users, and therefore can be used. Creating a .jceks file is possible on a 
host, 
`jceks://file/tmp/test.jceks`, whereas on HDFS the user may have to connect 
to hdfs and then create the file such as 
`jceks://h...@nn1.example.com/my/path/test.jceks`, when the password is being 
stored on hdfs. 

At this point in time we have solved the problem for not storing passwords 
in Shiro.ini, which can be stored at the zeppelin host itself. However we have 
to improve this solution and make it work for the rest of the use cases and 
that's where Credential API comes into play. 

Credential API is a generic solution which allows users to create password 
files for both of the cases - on the host and hdfs. Needless to point out that 
this has been used across Knox, which is a good standard for security. 

Please let me know if this makes sense, or if you have any more questions. 



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1301
  
Merging if there's 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 encrypting the plain text password. I don't 
know what the relationship is from HDFS. I think you should describe why you 
must use hadoop-common in details.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 conventions get stored on HDFS, where we will 
need a simplified, credential API, and therefore the choice is to use the 
credential API. Also this is pretty standard implementation in the Big Data 
world. 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1200: [ZEPPELIN-1191] Supported legacy way to run par...

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 ]

2016-08-22 Thread onkarshedge
Github user onkarshedge commented on the issue:

https://github.com/apache/zeppelin/pull/989
  
@bzz I found this 
[documentation](http://docs.ipfs.apiary.io/#reference/version/version/version) 
and there would be no need for dependency. I have used apache-httpcomponents 
httpclient.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1323
  
The codes look simple. @cloverhearts Can you help me review it?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
@prabhjyotsingh  I agree @echarles's idea. Interpreter tries to find hadoop 
dependencies first and if it passes, it uses `doAs`. Otherwise, let's talk 
about how to do it.  How do you think of it?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 support `jceks`? I've googled it and 
found some examples. Basically, `jceks` would be a good candidate for making 
password secured.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1283
  
@beriaanirudh You need to change more where you don't use 'diamond 
operator'. Can you fix them?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
LGTM except minor comment.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@astroshim I've also thought the interpreter `restart` button in separated 
page quite uncomfortable. This PR can be a good start for the better UX i think 
:)
I tested your patch and it almost works well except one thing. 

>can only restart `scoped` and `isolated` interpreter.

Since you said like above in the description,  I edited `Spark` interpreter 
as `shared` mode in the interpreter menu and restarted with your new 
interpreter binding UI. I expected it failed with some error messages, but it 
restarted. Did I miss sth? 



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
The CI build is green refer to 
https://travis-ci.org/apache/zeppelin/builds/154134031

Could someone please review this?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1354) Inject Services

2016-08-22 Thread Eric Charles (JIRA)
Eric Charles created ZEPPELIN-1354:
--

 Summary: Inject Services
 Key: ZEPPELIN-1354
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1354
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Eric Charles
Assignee: Eric Charles


Services like authentication, authorization, security filters... should be 
injected rather than manually instanciated.

This will allow to give central + finer control and abstracting the functionaly 
in interface.

For this, I propose to use Guice as injection framework.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1343: ZEPPELIN-1328 - z.show in python interpreter does not ...

2016-08-22 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1343
  
There actually are tests, but we do not have CI profiles with both Python2 
and Python3 yet. I think, I will take a look into creating one soon. 
Though @felixcheung is right, there are no Integration Tests for Python 
interpreter yet, the one that might verify that image appears in the paragraph. 
It would be nice to have one, but I guess it's a bit out of scope for this PR.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1284
  
As before, I think it will be more intuitive if in a paragraph, the user 
types:
```
%hadoop1
spark.
```
And to autocomplete and the popups says `hadoop1`?
Yes, I agree the completion is for "Spark stuff" but completion result 
might include items that are interpreter instance/state specific things, like 
local variables and so on?

That said, I don't feel strongly about this so I'll let others to comment.



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1353) java.lang.VerifyError for spark interpreter

2016-08-22 Thread hy5446 (JIRA)
hy5446 created ZEPPELIN-1353:


 Summary: java.lang.VerifyError for spark interpreter
 Key: ZEPPELIN-1353
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1353
 Project: Zeppelin
  Issue Type: Bug
Reporter: hy5446


I have just cloned the git repo and compiled zeppelin to try it out. 
Loosely following this guide: 
https://www.mapr.com/blog/building-apache-zeppelin-mapr-using-spark-under-yarn

git rev-parse HEAD
54d435303c58739de6ca57144f63532637cd291e

mvn package -Pbuild-distr -Pmapr51 -Pyarn -Pspark-1.6 -Pscala-2.10 -Phadoop-2.7 
-Pscalding -DskipTests

then, i edited the spark interpreter to change "master" to "yarn"
I tried a very simple program in my notebook:

%spark 
println("hello");

which failed and in the log I see this exception (see below)
can you help me to figure out what's wrong?

 INFO [2016-08-22 14:54:47,016] ({pool-1-thread-5} Logging.scala[logInfo]:58) - 
Successfully started service 'sparkDriver' on port 37520.
 INFO [2016-08-22 14:54:47,118] 
({sparkDriverActorSystem-akka.actor.default-dispatcher-3} 
Slf4jLogger.scala[applyOrElse]:80) - Slf4jLogger started
 INFO [2016-08-22 14:54:47,122] 
({sparkDriverActorSystem-akka.actor.default-dispatcher-7} 
Slf4jLogger.scala[apply$mcV$sp]:74) - Starting remoting
ERROR [2016-08-22 14:54:47,127] 
({sparkDriverActorSystem-akka.actor.default-dispatcher-7} 
Slf4jLogger.scala[apply$mcV$sp]:66) - Uncaught fatal error from thread 
[sparkDriverActorSystem-akka.remote.default-remote-dispatcher-8] shutting down 
ActorSystem [sparkDriverActorSystem]
java.lang.VerifyError: (class: 
org/jboss/netty/channel/socket/nio/NioWorkerPool, method: createWorker 
signature: 
(Ljava/util/concurrent/Executor;)Lorg/jboss/netty/channel/socket/nio/AbstractNioWorker;)
 Wrong return type in function
at 
akka.remote.transport.netty.NettyTransport.(NettyTransport.scala:283)
at 
akka.remote.transport.netty.NettyTransport.(NettyTransport.scala:240)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at 
akka.actor.ReflectiveDynamicAccess$$anonfun$createInstanceFor$2.apply(DynamicAccess.scala:78)
at scala.util.Try$.apply(Try.scala:161)
at 
akka.actor.ReflectiveDynamicAccess.createInstanceFor(DynamicAccess.scala:73)
at 
akka.actor.ReflectiveDynamicAccess$$anonfun$createInstanceFor$3.apply(DynamicAccess.scala:84)
at 
akka.actor.ReflectiveDynamicAccess$$anonfun$createInstanceFor$3.apply(DynamicAccess.scala:84)
at scala.util.Success.flatMap(Try.scala:200)
at 
akka.actor.ReflectiveDynamicAccess.createInstanceFor(DynamicAccess.scala:84)
at akka.remote.EndpointManager$$anonfun$9.apply(Remoting.scala:711)
at akka.remote.EndpointManager$$anonfun$9.apply(Remoting.scala:703)
at 
scala.collection.TraversableLike$WithFilter$$anonfun$map$2.apply(TraversableLike.scala:722)
at scala.collection.Iterator$class.foreach(Iterator.scala:727)
at scala.collection.AbstractIterator.foreach(Iterator.scala:1157)
at scala.collection.IterableLike$class.foreach(IterableLike.scala:72)
at scala.collection.AbstractIterable.foreach(Iterable.scala:54)
at 
scala.collection.TraversableLike$WithFilter.map(TraversableLike.scala:721)
at 
akka.remote.EndpointManager.akka$remote$EndpointManager$$listens(Remoting.scala:703)
at 
akka.remote.EndpointManager$$anonfun$receive$2.applyOrElse(Remoting.scala:491)
at akka.actor.Actor$class.aroundReceive(Actor.scala:467)
at akka.remote.EndpointManager.aroundReceive(Remoting.scala:394)
at akka.actor.ActorCell.receiveMessage(ActorCell.scala:516)
at akka.actor.ActorCell.invoke(ActorCell.scala:487)
at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:238)
at akka.dispatch.Mailbox.run(Mailbox.scala:220)
at 
akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec(AbstractDispatcher.scala:397)
at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
at 
scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
at 
scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
at 
scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)
 INFO [2016-08-22 14:54:47,130] 
({sparkDriverActorSystem-akka.actor.default-dispatcher-4} 
Slf4jLogger.scala[apply$mcV$sp]:74) - Shutting down remote daemon.
 INFO [2016-08-22 14:54:47,130] 
({sparkDriverActorSystem-akka.actor.default-dispatcher-4} 
Slf4jLogger.scala[apply$mcV$sp]:74) - Remote daemon shut down; 

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
Yes, you are right, let me do it right away.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread mwkang
Github user mwkang commented on the issue:

https://github.com/apache/zeppelin/pull/1350
  
@felixcheung Okay! I will add unit test. Thanks for your comment!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1350
  
is it possible to add some unit test for this?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?)

### What is this PR for?
This handles the NPE when the input json is empty for the interpreter 
setting POST request.

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

### Todos
NA

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

### How should this be tested?
When empty json is sent for interpreter setting POST request, 400 status 
code should be returned.

### 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/kavinkumarks/zeppelin 
zeppelin-728-fix-NPE-intepreter-setting-post

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

https://github.com/apache/zeppelin/pull/1345.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 #1345


commit ab7f9e8029e1e272810bd14b7fcbf14ea2cb62a8
Author: Kavin Kumar 
Date:   2016-08-19T07:14:11Z

Handled NPE when the json is empty for interpreter setting POST request
and corrected the json in the REST API doc.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1347
  
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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1343: ZEPPELIN-1328 - z.show in python interpreter does not ...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1343
  
LGTM. is it easy to add automated unit test for this?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1338: Zeppelin 1307 - Implement notebook revision in Zeppeli...

2016-08-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1338
  
somehow i keep getting this error when doing checkpoint: 
```
ERROR [2016-08-22 14:33:04,618] ({qtp787867107-76} 
NotebookServer.java[onMessage]:257) - Can't handle message
java.lang.NullPointerException
at org.eclipse.jetty.client.HttpClient.send(HttpClient.java:521)
at org.eclipse.jetty.client.HttpRequest.send(HttpRequest.java:694)
at org.eclipse.jetty.client.HttpRequest.send(HttpRequest.java:678)
at 
org.apache.zeppelin.notebook.repo.zeppelinhub.rest.ZeppelinhubRestApiHandler.sendToZeppelinHub(ZeppelinhubRestApiHandler.java:157)
at 
org.apache.zeppelin.notebook.repo.zeppelinhub.rest.ZeppelinhubRestApiHandler.asyncPutWithResponseBody(ZeppelinhubRestApiHandler.java:125)
at 
org.apache.zeppelin.notebook.repo.zeppelinhub.ZeppelinHubRepo.checkpoint(ZeppelinHubRepo.java:201)
at 
org.apache.zeppelin.notebook.repo.NotebookRepoSync.checkpoint(NotebookRepoSync.java:334)
at 
org.apache.zeppelin.notebook.Notebook.checkpointNote(Notebook.java:358)
at 
org.apache.zeppelin.socket.NotebookServer.checkpointNotebook(NotebookServer.java:1182)
at 
org.apache.zeppelin.socket.NotebookServer.onMessage(NotebookServer.java:233)
at 
org.apache.zeppelin.socket.NotebookSocket.onWebSocketText(NotebookSocket.java:56)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextMessage(JettyListenerEventDriver.java:128)
at 
org.eclipse.jetty.websocket.common.message.SimpleTextMessage.messageComplete(SimpleTextMessage.java:69)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.appendMessage(AbstractEventDriver.java:65)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextFrame(JettyListenerEventDriver.java:122)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.incomingFrame(AbstractEventDriver.java:161)
at 
org.eclipse.jetty.websocket.common.WebSocketSession.incomingFrame(WebSocketSession.java:309)
at 
org.eclipse.jetty.websocket.common.extensions.ExtensionStack.incomingFrame(ExtensionStack.java:214)
at 
org.eclipse.jetty.websocket.common.Parser.notifyFrame(Parser.java:220)
at org.eclipse.jetty.websocket.common.Parser.parse(Parser.java:258)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.readParse(AbstractWebSocketConnection.java:632)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillable(AbstractWebSocketConnection.java:480)
at 
org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
```
not sure what's the problem though, let me know if doing something wrong


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


RE: Issues in Zeppelin 0.6.0

2016-08-22 Thread Nagasravanthi, Valluri
Hi Jeff,

We are still trying to sort out the issue that we highlighted before, so it’d 
be great if you could give your insight as soon as possible so that we can try 
and tackle it in a suitable manner.

Thanks and Regards,

[Description: cid:image001.png@01D1EBF4.36D373B0]

Valluri Naga Sravanthi| On assignment to PfizerWorks
Cell: +91 9008412366
Email: pfizerwor...@pfizer.com; 
valluri.nagasravan...@pfizer.com
Website: http://pfizerWorks.pfizer.com



……
From: Nagasravanthi, Valluri
Sent: Friday, August 19, 2016 8:39 AM
To: 'Jeff Zhang'
Cc: dev@zeppelin.apache.org; d...@zeppelin.incubator.apache.org; 
us...@zeppelin.apache.org; Jaisankar, Saurabh; Sundararajan, Pranav
Subject: RE: Issues in Zeppelin 0.6.0

We upgraded Ambari Cluster  while we were working with Zeppelin 0.5.6. At that 
point, we were using Spark 1.5.2. After the upgrade of Ambari, the Spark 
version got upgraded to 1.6. Post that we upgraded Zeppelin to 0.6.0.

From: Jeff Zhang [mailto:zjf...@gmail.com]
Sent: Friday, August 19, 2016 7:48 AM
To: Nagasravanthi, Valluri
Cc: dev@zeppelin.apache.org; 
d...@zeppelin.incubator.apache.org; 
us...@zeppelin.apache.org; Jaisankar, 
Saurabh; Sundararajan, Pranav
Subject: Re: Issues in Zeppelin 0.6.0

Do you change spark version when upgrading zeppelin ?

On Fri, Aug 19, 2016 at 7:36 PM, Nagasravanthi, Valluri 
> 
wrote:

Hi,

Actually, when we worked on Zeppelin 0.5.6, we used to use sqlContext itself 
for reading hive files, registering spark dataframes as temp tables and 
eventualy processing queries on the temp tables.

The issue which I am facing on Zeppelin 0.6.0 right now  is that even though I 
am able to execute “SELECT”  query on the temptables but I am not able to 
execute DDL statements like Create/Drop tables using temptables derived from 
the hive files.

To elaborate it further:
When I write the following codes, they work fine. PFB the sample codes:

  *   sqlContext.sql(“select * from table name”)
  *   sqlContext.sql(“Select count(column_name) from table_name”)
But when I write the following codes, it fails to execute:

  *   sqlContext.sql(“create tablename as select * from tablename_1”)
  *   sqlContext.sql(“drop if exists tablename”)

I am getting the error : java.lang.RuntimeException: [1.1] failure: ``with'' 
expected but identifier drop found :
I even tried using sql interpreter.

PFB the code for it:
%sql
Drop if exists tablename
I am getting the same error again: : java.lang.RuntimeException: [1.1] failure: 
``with'' expected but identifier drop found :

Thanks a lot for being so responsive. Hope to fix this issue soon with your 
help.

Thanks and Regards,

[Description: cid:image001.png@01D1EBF4.36D373B0]

Valluri Naga Sravanthi| On assignment to PfizerWorks
Cell: +91 9008412366
Email: pfizerwor...@pfizer.com; 
valluri.nagasravan...@pfizer.com
Website: http://pfizerWorks.pfizer.com



……


From: Jeff Zhang [mailto:zjf...@gmail.com]


Sent: Friday, August 19, 2016 6:49 AM
To: Sundararajan, Pranav
Cc: dev@zeppelin.apache.org; 
d...@zeppelin.incubator.apache.org; 
us...@zeppelin.apache.org; Jaisankar, 
Saurabh; Nagasravanthi, Valluri

Subject: Re: Issues in Zeppelin 0.6.0

Sorry, you should use sqlContext

On Fri, Aug 19, 2016 at 6:11 PM, Sundararajan, Pranav 
> wrote:
Hi,

We tried “hiveContext” instead of “HiveContext” but it didn’t work out.
Please find below the code and the error log:

import org.apache.spark.sql.hive.HiveContext
val df_test_hive = hiveContext.read.parquet("/location/hivefile")

import org.apache.spark.sql.hive.HiveContext
:67: error: not found: value hiveContext
val df_test_hive = hiveContext.read.parquet("/location/hivefile")

It seems hiveContext is not an object of HiveContext package.
Could you please suggest if there is any other way out?


Regards,






[Description: cid:image001.png@01D1C173.60CDE0C0]

Pranav Sundararajan| On assignment to PfizerWorks
Cell: +91 9008412366
Email: pfizerwor...@pfizer.com; 
pranav.sundarara...@pfizer.com
Website: 

[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
... and make ZEPPELIN-1320 a subtask of ZEPPELIN-1337

?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
To make ZEPPELIN-1337 Umbrella for multiple user support for zeppelin more 
readable, should we rename the following:

- ZEPPELIN-1340: "Run Hadoop-based interpreter process on Kerberos as web 
front end user"
- ZEPPELIN-1320:  "Run zeppelin interpreter process as web front end user"


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
Github user astroshim closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1291]Tagging meta value on interpreter completion.

### What is this PR for?
This PR is for showing the meta tag on auto completion.


### What type of PR is it?
Improvement


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


### How should this be tested?
Please refer to screenshot.


### Screenshots (if appropriate)
- before 1 

![image](https://cloud.githubusercontent.com/assets/3348133/17408054/a72929f6-5aa4-11e6-9b12-a3f0fb6dfda8.png)

- before 2

![image](https://cloud.githubusercontent.com/assets/3348133/17408076/bd068426-5aa4-11e6-8129-72570002807e.png)


- after

![after](https://cloud.githubusercontent.com/assets/3348133/17408100/d8104568-5aa4-11e6-85e8-d4717edc8041.gif)


### 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/astroshim/zeppelin ZEPPELIN-1291

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

https://github.com/apache/zeppelin/pull/1284.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 #1284


commit 4c49432e43c4963b5266044a82d1002cd12931f2
Author: astroshim 
Date:   2016-08-04T10:44:02Z

add meta tag on auto completion

commit 78c853aa87a45e52e7e50c6965135819ace3628d
Author: astroshim 
Date:   2016-08-05T05:21:58Z

thrift 0.9.2 and add meta description

commit ce86d78c8d3edd2830c31264e836e7218c0b0253
Author: astroshim 
Date:   2016-08-18T10:12:25Z

rebase




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-22 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1284
  
re-trigger CI


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
@echarles , Yes agreed, will need to update in doc, and a extra toolbar 
near the check box where user can enable User Impersonate.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-08-22 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
Whatever `su` or `ssh` is used, I feel the main trick is the user 
provisioning on the host running the interpreter. Until now, the shiro 
authentication system had no impact on the user provisioning. This PR changes 
this.

I guess we all agree and are aware that adding user `foo` to shiro.ini, and 
enabling impersonation, will require to run `adduser foo` manually.

We should make this clear in the doc but also stress it in the UI (with a 
hover, or a clear text/link near the  User Impersonate.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
The CI build is green refer to 
https://travis-ci.org/apache/zeppelin/builds/154093445

Could someone please review this?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?)

### What is this PR for?
This handles the NPE when the input json is empty for the interpreter 
setting POST request.

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

### Todos
NA

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

### How should this be tested?
When empty json is sent for interpreter setting POST request, 400 status 
code should be returned.

### 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/kavinkumarks/zeppelin 
zeppelin-728-fix-NPE-intepreter-setting-post

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

https://github.com/apache/zeppelin/pull/1345.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 #1345


commit ab7f9e8029e1e272810bd14b7fcbf14ea2cb62a8
Author: Kavin Kumar 
Date:   2016-08-19T07:14:11Z

Handled NPE when the json is empty for interpreter setting POST request
and corrected the json in the REST API doc.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1341: [ZEPPELIN-1162] Fix rawType in NotebookRestApi

2016-08-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-1313] NullPointerException when using Clone notebook REST API

### What is this PR for?
This fixes when the input json is empty for the clone notebook REST API and 
for this case the default name will be populated.

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

### Todos
NA

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

### How should this be tested?
When the input json is empty for the clone notebook REST API, the response 
should be 201 with the default name populated.

### 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/kavinkumarks/zeppelin 
zeppelin-1313-fix-NPE-clone-notebook

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

https://github.com/apache/zeppelin/pull/1348.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 #1348






---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?)

### What is this PR for?
This handles the NPE when the input json is empty for the interpreter 
setting POST request.

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

### Todos
NA

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

### How should this be tested?
When empty json is sent for interpreter setting POST request, 400 status 
code should be returned.

### 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/kavinkumarks/zeppelin 
zeppelin-728-fix-NPE-intepreter-setting-post

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

https://github.com/apache/zeppelin/pull/1345.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 #1345


commit ab7f9e8029e1e272810bd14b7fcbf14ea2cb62a8
Author: Kavin Kumar 
Date:   2016-08-19T07:14:11Z

Handled NPE when the json is empty for interpreter setting POST request
and corrected the json in the REST API doc.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1340: [MINOR][DOC] Update available interpreters' image in i...

2016-08-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1340
  
@anthonycorbacho Updated the image as you said. You can check `before` & 
`after` in the PR description. 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

2016-08-22 Thread beriaanirudh
Github user beriaanirudh commented on the issue:

https://github.com/apache/zeppelin/pull/1283
  
@corneadoug @jongyoul @bzz this is ready for review.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1348
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1348: [ZEPPELIN-1313] NullPointerException when using...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-1313] NullPointerException when using Clone notebook REST API

### What is this PR for?
This fixes when the input json is empty for the clone notebook REST API and 
for this case the default name will be populated.

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

### Todos
NA

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

### How should this be tested?
When the input json is empty for the clone notebook REST API, the response 
should be 201 with the default name populated.

### 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/kavinkumarks/zeppelin 
zeppelin-1313-fix-NPE-clone-notebook

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

https://github.com/apache/zeppelin/pull/1348.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 #1348


commit 19aa878e0183530ef113790e6e1275efa8d59fde
Author: Kavin Kumar 
Date:   2016-08-19T12:51:43Z

Fixed NPE when the input json is empty for clone notebook REST API and
set the default name for the case.Added test cases too.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks closed the pull request at:

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


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1345: [ZEPPELIN-728] Can't POST interpreter setting (CorsFil...

2016-08-22 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1345
  
Reopening to trigger the CI build.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1345: [ZEPPELIN-728] Can't POST interpreter setting (...

2016-08-22 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request:

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

[ZEPPELIN-728] Can't POST interpreter setting (CorsFilter?)

### What is this PR for?
This handles the NPE when the input json is empty for the interpreter 
setting POST request.

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

### Todos
NA

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

### How should this be tested?
When empty json is sent for interpreter setting POST request, 400 status 
code should be returned.

### 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/kavinkumarks/zeppelin 
zeppelin-728-fix-NPE-intepreter-setting-post

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

https://github.com/apache/zeppelin/pull/1345.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 #1345


commit ab7f9e8029e1e272810bd14b7fcbf14ea2cb62a8
Author: Kavin Kumar 
Date:   2016-08-19T07:14:11Z

Handled NPE when the json is empty for interpreter setting POST request
and corrected the json in the REST API doc.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---