Re: [VOTE] Creating feature branch for zeppelin new ui initiative

2019-09-04 Thread Prabhjyot Singh
+1 Excellent, this was long overdue. On Thu, 5 Sep 2019 at 08:43, Jeff Zhang wrote: > Sure, we should definitely run CI before merging such large feature. > > Jongyoul Lee 于2019年9月5日周四 上午11:01写道: > > > Hello, > > > > Additionally, if we make a PR and check the CI properly before we merge > >

[GitHub] [zeppelin] ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export

2019-09-04 Thread GitBox
ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export URL: https://github.com/apache/zeppelin/pull/2649#issuecomment-528188568 @zjffdu Cool! But I haven't resolved the issues yet. I'll let you know when they're resolved. I just did a random commit to check the CI

[GitHub] [zeppelin] zjffdu commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export

2019-09-04 Thread GitBox
zjffdu commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export URL: https://github.com/apache/zeppelin/pull/2649#issuecomment-528187541 Thanks @ShamsUlAzeem Seems the CI is still running, let's wait for the CI result.

[GitHub] [zeppelin] ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export

2019-09-04 Thread GitBox
ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export URL: https://github.com/apache/zeppelin/pull/2649#issuecomment-528185967 @zjffdu yup I configured it a long time ago and it's working now as well. I'll do a couple of changes and run some tests before notifying you

[GitHub] [zeppelin] zjffdu commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export

2019-09-04 Thread GitBox
zjffdu commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export URL: https://github.com/apache/zeppelin/pull/2649#issuecomment-528184942 @ShamsUlAzeem Have you configured your travis to enable the CI ? https://zeppelin.apache.org/contribution/contributions.html#continuous-integration

Re: [VOTE] Creating feature branch for zeppelin new ui initiative

2019-09-04 Thread Jeff Zhang
Sure, we should definitely run CI before merging such large feature. Jongyoul Lee 于2019年9月5日周四 上午11:01写道: > Hello, > > Additionally, if we make a PR and check the CI properly before we merge > that branch to master, I agree to make a new branch, and I'll help to > change backend sides as well.

Re: [VOTE] Creating feature branch for zeppelin new ui initiative

2019-09-04 Thread Jongyoul Lee
Hello, Additionally, if we make a PR and check the CI properly before we merge that branch to master, I agree to make a new branch, and I'll help to change backend sides as well. JL On Thu, Sep 5, 2019 at 11:33 AM Jeff Zhang wrote: > Hi all, > > There's one thread [1] discussing about the new

[VOTE] Creating feature branch for zeppelin new ui initiative

2019-09-04 Thread Jeff Zhang
Hi all, There's one thread [1] discussing about the new zeppelin ui initiative. It looks like the community have reach consensus on this. Considering this is a large effort and most of work is on frontend, I would suggest to create a new feature branch for this initiative. So that the

[GitHub] [zeppelin] santosh1994 opened a new pull request #3438: [ZEPPELIN-4320] fix packages to bundle list

2019-09-04 Thread GitBox
santosh1994 opened a new pull request #3438: [ZEPPELIN-4320] fix packages to bundle list URL: https://github.com/apache/zeppelin/pull/3438 ### What is this PR for? Enabled Helium packages are not getting installed/bundled on zeppelin startup. Instead getting bundled later on paragraph

[GitHub] [zeppelin] ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export

2019-09-04 Thread GitBox
ShamsUlAzeem commented on issue #2649: [ZEPPELIN-3033] IPYNB import/export URL: https://github.com/apache/zeppelin/pull/2649#issuecomment-527828390 @zjffdu ah alright. on it This is an automated message from the Apache Git

[GitHub] [zeppelin] santosh1994 opened a new pull request #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic

2019-09-04 Thread GitBox
santosh1994 opened a new pull request #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic URL: https://github.com/apache/zeppelin/pull/3437 ### What is this PR for? Spell interpreter returns incorrect results when paragraph having magic in between text, with current approach

[jira] [Created] (ZEPPELIN-4320) Enabled Helium packages are not getting bundled on zeppelin start

2019-09-04 Thread santosh k (Jira)
santosh k created ZEPPELIN-4320: --- Summary: Enabled Helium packages are not getting bundled on zeppelin start Key: ZEPPELIN-4320 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4320 Project:

[jira] [Created] (ZEPPELIN-4319) Spell Interpreter returns incorrect results when magic is present in paragraph text

2019-09-04 Thread santosh k (Jira)
santosh k created ZEPPELIN-4319: --- Summary: Spell Interpreter returns incorrect results when magic is present in paragraph text Key: ZEPPELIN-4319 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4319