You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by necosta <gi...@git.apache.org> on 2018/01/24 15:54:32 UTC
[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API
GitHub user necosta opened a pull request:
https://github.com/apache/zeppelin/pull/2745
[ZEPPELIN-3189] NPE on paragraph run via API
### What is this PR for?
Bugfix over NPE when running REST API command to run paragraph synchronously.
### What type of PR is it?
[Bug Fix]
### Todos
* [ ] - Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3189
### How should this be tested?
1. Build Zep (mvn clean package -DskipTests)
2. Start Zep
3. Run curl command in shell (curl -i -X POST http://localhost:8080/api/notebook/run/{noteId}/{paragraphId}
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? N
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nokia/zeppelin zeppelin3189
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2745.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 #2745
----
commit 9d1e2a569c914d94b9d4dfa7bb98f7c7301fbe9e
Author: Nelson Costa <ne...@...>
Date: 2018-01-24T15:46:14Z
[ZEPPELIN-3189] NPE on paragraph run via API
----
---
[GitHub] zeppelin issue #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2745
Merge to master if no further comment
---
[GitHub] zeppelin issue #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by necosta <gi...@git.apache.org>.
Github user necosta commented on the issue:
https://github.com/apache/zeppelin/pull/2745
ping @zjffdu , @Leemoonsoo , @1ambda
Thanks
---
[GitHub] zeppelin issue #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by necosta <gi...@git.apache.org>.
Github user necosta commented on the issue:
https://github.com/apache/zeppelin/pull/2745
Closed/Reopened to refresh GitHub...
Green CI: https://travis-ci.org/nokia/zeppelin/builds/332846355
---
[GitHub] zeppelin issue #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2745
Thanks @necosta for the fix. LGTM
---
[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by necosta <gi...@git.apache.org>.
Github user necosta closed the pull request at:
https://github.com/apache/zeppelin/pull/2745
---
[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by necosta <gi...@git.apache.org>.
GitHub user necosta reopened a pull request:
https://github.com/apache/zeppelin/pull/2745
[ZEPPELIN-3189] NPE on paragraph run via API
### What is this PR for?
Bugfix over NPE when running REST API command to run paragraph synchronously.
### What type of PR is it?
[Bug Fix]
### Todos
* [ ] - Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3189
### How should this be tested?
1. Build Zep (mvn clean package -DskipTests)
2. Start Zep
3. Run curl command in shell (curl -i -X POST http://localhost:8080/api/notebook/run/{noteId}/{paragraphId}
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? N
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nokia/zeppelin zeppelin3189
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2745.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 #2745
----
commit 9d1e2a569c914d94b9d4dfa7bb98f7c7301fbe9e
Author: Nelson Costa <ne...@...>
Date: 2018-01-24T15:46:14Z
[ZEPPELIN-3189] NPE on paragraph run via API
----
---
[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2745
---
[GitHub] zeppelin issue #2745: [ZEPPELIN-3189] NPE on paragraph run via API
Posted by zjffdu <gi...@git.apache.org>.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2745
LGTM
---