Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1861
LGTM.
what is the bug in LivySparkSQLInterpreter - should we have a JIRA for it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
Tested, 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 i
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1880
ZEPPELIN-1918. Fix build with Spark 2.1.0
### What is this PR for?
It's my misunderstanding of `SPARK_VER` in travis. It is only used for
downloading spark distribution. We need to specify spa
GitHub user cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1879
[HOTFIX : ZEPPELIN-1932] paragraph blur error
### What is this PR for?
When one or more hidden editors are present, clicking on the editor will
cause a blur error.
This means that
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1879
---
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 featur
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1850
---
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 e
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1845
@astroshim do you have some cycle to 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 ha
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1787
Sure, let me fix 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 s
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1787
Let's get this fix ASAP - we are going to cut 0.7 release in 2 days
---
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 pro
Hi,
It's been almost 3 month since we did 0.6.2 release and there were number
of improvements and bug fixes.
Most of the items that we planned for 0.7.0 release [1] are ready
so I am planning to cut branch-0.7 on 10th Jan 11pm(PST).
After branch cut, bug fixes, documentation patches and issues li
GitHub user cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1879
[HOTFIX : ZEPPELIN-1932] paragraph blur error
### What is this PR for?
When one or more hidden editors are present, clicking on the editor will
cause a blur error.
This means that wh
CloverHearts created ZEPPELIN-1932:
--
Summary: When one or more hidden editors are present, clicking on
the editor will cause a blur error.
Key: ZEPPELIN-1932
URL: https://issues.apache.org/jira/browse/ZEPPELIN-19
GitHub user soralee reopened a pull request:
https://github.com/apache/zeppelin/pull/1804
[ZEPPELIN-1578] notes list in customizing zeppelin homepage isn't working
### What is this PR for?
Customizing the Zeppelin Homepage to show the notebook list does not work
as [0.6.2
docum
Github user soralee closed the pull request at:
https://github.com/apache/zeppelin/pull/1804
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1876
LGTM and merge if no more comments.
---
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
en
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1865
---
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 e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1846
[AuthenticationIT.java](https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/test/java/org/apache/zeppelin/integration/AuthenticationIT.java)
is test that uses zeppelin with shiro
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1846
@Leemoonsoo Could you point me the right place to put this unit test? I
think this may need to enable shiro, but I didn't find any test with shiro
enabled.
---
If your project is set up for it,
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1787
ping @ejono Do you want to contribute it ? If not, I will fix 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 do
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1838
---
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 e
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1849
---
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 e
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1870
---
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 e
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1842
Let me review this PR. I wil comment again :)
---
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 fea
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1842
Awesome! 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
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1877
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1877
LGTM and merge to master if there're no more discussions
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1875
@Leemoonsoo I moved all font-related licenses from `bin_license` to
`/LICENSE`
Aldo remove rat exclusion in `zeppeiln-web/pom.xml` for `google-fonts.css`
because it is managed by us now.
---
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1877
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
GitHub user cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1878
add yarn_error.log in gitignore
### What is this PR for?
Currently zeppelin is built from yarn.
Therefore, we need to add a log to gitignore.
### What type of PR is it?
I
CloverHearts created ZEPPELIN-1931:
--
Summary: we should be add yarn_error.log on gitignore.
Key: ZEPPELIN-1931
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1931
Project: Zeppelin
Issu
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1789
Tested and LGTM. Thanks @cuspymd for the nice optimization!
Merge to master if no more discussions.
---
If your project is set up for it, you can reply to this email and have your
repl
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1852
Thanks @1ambda for the fix.
LGTM and merge if no more comments!
---
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 proje
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1877
[ZEPPELIN-1930](HotFix) PythonInterpreter syntax error.
### What is this PR for?
This PR fixes syntax error of PythonInterpreter.
### What type of PR is it?
Bug Fix
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1877
---
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 i
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1877
@Leemoonsoo I added testcase, could you help to 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 h
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1877
@agoodm pyspark interpreter is working well.
---
If your project is set up fo
Github user AlexanderShoshin commented on the issue:
https://github.com/apache/zeppelin/pull/1758
I've added a new commit to convert the notebook to 0.7.0 format because I
found that
[9013620](https://github.com/apache/zeppelin/pull/1758/commits/9013620e1c835c66a849467e1ea15d43fa2a778
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1875
Honestly, I want to compole web in 30 secs. Do you make it possible?
Kkkk
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1877
I faced the same problem too. Thanks for fix!
---
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 feat
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1868
@Leemoonsoo Thanks for review!
If https://github.com/apache/zeppelin/pull/1874 is merged soon, I can
rewrite the whole `conda env *` commands based on #1874
since current implementati
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1875
@jongyoul small amount of :)
if you want to reduce time for `npm install`, you can use `yarn install`
instead of. It will decrease 20~50 secs.
And https://github.com/apache/zeppelin/pu
Github user agoodm commented on the issue:
https://github.com/apache/zeppelin/pull/1877
I am on vacation right now so I can't thoroughly examine this, but have you
confirmed that this isn't a problem for the pyspark interpreter?
---
If your project is set up for it, you can reply to
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1864
---
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 featur
GitHub user cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1864
[ZEPPELIN-1921] missing close in closeAndRemoveInterpreterGroup method
### What is this PR for?
The problem is that some code in the closeAndRemoveInterpreterGroup method
of Interprete
Github user bitchelov commented on the issue:
https://github.com/apache/zeppelin/pull/1747
@lresende is it right?
---
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 wish
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1846
@zjffdu Thanks for the patch!
I also think this PR deserves a unittest to make sure correct restart
behavior, like @felixcheung mentioned, if it is not too much difficult.
---
If your
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1877
Thanks for the fix. \cc @agoodm
How about add an unittest for this case in
https://github.com/apache/zeppelin/blob/master/python/src/test/java/org/apache/zeppelin/python/PythonInterpre
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1860
LGTM and merge to master if no further discussions
---
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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1877
[ZEPPELIN-1930](HotFix) PythonInterpreter syntax error.
### What is this PR for?
This PR fixes syntax error of PythonInterpreter.
### What type of PR is it?
Bug Fix
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1812
---
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 e
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1834
@AhyoungRyu I updated the filter of Navbar.
But, the folder of notes in navbar don't have previous status. Because If
using the hidden value of the note, it works together both the filter in t
Shim HyungSung created ZEPPELIN-1930:
Summary: SyntaxError on PythonInterpreter
Key: ZEPPELIN-1930
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1930
Project: Zeppelin
Issue Type: B
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1872
You don't have to make two PRs for adopting multiple branches. We can merge
it into several branches with one commit if there's no conflict. Can you please
close this PR?
---
If your project is
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1861
Tested locally, looks good.
---
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
enable
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1864
---
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 featur
GitHub user cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1864
[ZEPPELIN-1921] missing close in closeAndRemoveInterpreterGroup method
### What is this PR for?
The problem is that some code in the closeAndRemoveInterpreterGroup method
of Interprete
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1875
@1ambda Does it reduce the build time of web module?
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1874
LGTM and merge to master if no further discussions.
---
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
Shim HyungSung created ZEPPELIN-1929:
Summary: need to fix the output format of PythonInterpreter
Key: ZEPPELIN-1929
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1929
Project: Zeppelin
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1868
Great improvement! Tested and LGTM.
Is it on purpose not supporting `conda env remove` ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1861
@felixcheung I have addressed the comments and also fix one bug in
LivySparkSQLInterpreter, and update the screenshot of testing in PR
description.
---
If your project is set up for it, you can
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1838
Thx for the fix. I will merge this one as temporary fix, but I will keep
the jira issue open until we come up with better solution.
---
If your project is set up for it, you can reply to this em
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1876
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 t
Github user AlexanderShoshin commented on the issue:
https://github.com/apache/zeppelin/pull/1758
Sorry, I was not able to work on this issue during last weeks.
@AhyoungRyu, I will make the corrections soon.
Thanks
---
If your project is set up for it, you can reply to this em
Github user ChrisAdvance commented on the issue:
https://github.com/apache/zeppelin/pull/1030
Does the "Connection refused" problem fixed yet?
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1875
@Leemoonsoo Thanks for detailed explanation. I will take a look.
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1875
This PR makes zeppelin source distribution package contains font files.
Therefore, related font license should be moved from
[`/zeppelin-distribution/src/bin-license/LICENSE`](https://github
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1838
@cloverhearts The last travis doesn't look related error.
---
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
68 matches
Mail list logo
