[GitHub] zeppelin pull request #2853: [ZEPPELIN-3306] Web e2e test fails

2018-03-08 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-3306] Web e2e test fails

### What is this PR for?
This is one of the flaky tests that fails with "should have a welcome 
message" because Z-Server did not start by the time test started executing.

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

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

### How should this be tested?
* CI should pass

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

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

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


commit 8742b1104f7d3671d33343006a25decf9618f79b
Author: Prabhjyot Singh 
Date:   2018-03-09T07:42:12Z

ZEPPELIN-3306: Web e2e test fails

Change-Id: If29e84e5ff2b3bf06dbf624d6baab87b6b9f38c4




---


[GitHub] zeppelin pull request #2749: [ZEPPELIN-3180] BUGFIX - save operation on inte...

2018-03-08 Thread asfgit
Github user asfgit closed the pull request at:

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


---


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

2018-03-08 Thread gss2002
Github user gss2002 closed the pull request at:

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


---


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

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/2851
  
@zjffdu thanks for the insight. It's setup and running now.


---


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

2018-03-08 Thread gss2002
GitHub user gss2002 reopened a pull request:

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

ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE

What is this PR for?
During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium 
and other functions leaving unusable notebooks.

What type of PR is it?
[Bug Fix]

Todos
 - Task

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

How should this be tested?
Manually tested using Import/Clone of Notebooks and attempt to adjust bound 
interpreters and tested using new unit test to clone notebooks to verify 
username is added during clone.

Screenshots (if appropriate)

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

Author: Greg Senia gse...@apache.org

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

$ git pull https://github.com/gss2002/zeppelin ZEPPELIN-3309

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

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


commit 091130073ee545a4f637d53708ee1f7165bddebc
Author: gss2002 
Date:   2018-03-08T22:06:06Z

ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE




---


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

2018-03-08 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2851
  
You didn't configure travis, see guidance here. 
http://zeppelin.apache.org/contribution/contributions.html#continuous-integration


---


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

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/2851
  
@zjffdu any chance you would possibly know why the commit wouldn't be found?

[2018-03-08 22:14:11] Can't find build for commit 
091130073ee545a4f637d53708ee1f7165bddebc from gss2002



---


[GitHub] zeppelin issue #2749: [ZEPPELIN-3180] BUGFIX - save operation on interpreter...

2018-03-08 Thread weand
Github user weand commented on the issue:

https://github.com/apache/zeppelin/pull/2749
  
*no more comment*, we really would like to see this feature in master and 
branch-0.8 branches


---


[GitHub] zeppelin issue #2817: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-08 Thread SleepyThread
Github user SleepyThread commented on the issue:

https://github.com/apache/zeppelin/pull/2817
  
I have created a [clean pull 
request](https://github.com/apache/zeppelin/pull/2852). Closing this pull 
request.  


---


[GitHub] zeppelin pull request #2817: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-08 Thread SleepyThread
Github user SleepyThread closed the pull request at:

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


---


[GitHub] zeppelin pull request #2852: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-08 Thread SleepyThread
GitHub user SleepyThread opened a pull request:

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

[ZEPPELIN-3264] Notebook Snapshot feature.

### What is this PR for?
Currently, a user can share the notebook with other users. There is a usage 
pattern where data analyst creates notebook report and then share with other 
stakeholders. Meanwhile, they would like to update/change the report and 
re-publish the results.  


We need a shareable url like,
 /notebook/{**Notebook ID**}/view/{**Snapshot Name**}

which can be shared to other users and can be updated with a logical 
sharable report. 


### What type of PR is it?
[ Feature | Documentation]

### Todos
* [ ] - CSS and assets changes. 

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

### How should this be tested?
* Create git based Notebook. 
* Create 1st commit. 
* Add a snapshot ID. 
* View the snapshot. 

### Screenshots (if appropriate)




![create_snapshot](https://user-images.githubusercontent.com/858038/36646998-ffca9ce6-1a76-11e8-9101-23a070a0b532.gif)



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

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

$ git pull https://github.com/SleepyThread/zeppelin 
ZEPPELIN-3264_notebook_snapshots

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

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


commit ae92c6e23064278ce5ac67669718c3aea927b1a2
Author: Akash 
Date:   2018-02-24T22:10:48Z

[ZEPPELIN-3264] Notebook Snapshot feature.




---


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

2018-03-08 Thread gss2002
GitHub user gss2002 opened a pull request:

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

ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE

What is this PR for?
During Import/Clone Paragraph set "user" to eliminate NPEs thrown in Helium 
and other functions leaving unusable notebooks.

What type of PR is it?
[Bug Fix]

Todos
 - Task

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

How should this be tested?
Manually tested using Import/Clone of Notebooks and attempt to adjust bound 
interpreters and tested using new unit test to clone notebooks to verify 
username is added during clone.

Screenshots (if appropriate)

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

Author: Greg Senia gse...@apache.org

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

$ git pull https://github.com/gss2002/zeppelin ZEPPELIN-3309

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

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


commit 091130073ee545a4f637d53708ee1f7165bddebc
Author: gss2002 
Date:   2018-03-08T22:06:06Z

ZEPPELIN-3309. Import/Clone doesn't set user in Paragraph causing NPE




---


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

2018-03-08 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/2849
  
Closing this pull request for now will re-open with a new one which 
includes formatting changes and corrections and test cases.


---


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

2018-03-08 Thread gss2002
Github user gss2002 closed the pull request at:

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


---


[jira] [Created] (ZEPPELIN-3311) Zeppelin should ask user to re-login after JWT cookie expires

2018-03-08 Thread Supreeth Sharma (JIRA)
Supreeth Sharma created ZEPPELIN-3311:
-

 Summary: Zeppelin should ask user to re-login after JWT cookie 
expires 
 Key: ZEPPELIN-3311
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3311
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-server
Affects Versions: 0.8.0
Reporter: Supreeth Sharma


Zeppelin should ask user to re-login after JWT cookie expires.

Steps to reproduce :
1) Enable SSO for zeppelin
2) Login to zeppelin using SSO.
3) Now wait till JWT expires(knoxsso.token.ttl under Advanced knoxsso-topology 
could be set to lower values to get a short lived cookie)
4) Refresh zeppelin. Zeppelin doesn't ask for re-login, though JWT is already 
expired and not valid anymore.



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


[jira] [Created] (ZEPPELIN-3310) Scio interpreter layout is broken

2018-03-08 Thread Satoshi Iijima (JIRA)
Satoshi Iijima created ZEPPELIN-3310:


 Summary: Scio interpreter layout is broken
 Key: ZEPPELIN-3310
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3310
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.3, 0.8.0
Reporter: Satoshi Iijima
 Attachments: scio_interpreter_layout_is_broken.png

Scio interpreter layout is broken. 

Screenshot is attached.

 



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