Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1861
The bug in LivySparkSQLInterpreter is caused by this PR.
LivySparkSQLInterpreter delegate LivySparkInterpreter to generate
InterpreterResult and then convert it to table type. So here since we make
Github user ejono commented on the issue:
https://github.com/apache/zeppelin/pull/1787
Sorry, I was not able to work on this over the past few days. Thank you
very much for fixing it, @zjffdu!
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1825
I updated this docs page again since REST API operations need to be listed
as CRUD order. And removed unimplemented response code info as well. Please see
the updated screenshot images :)
---
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1825
Nice improvement, 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 wishe
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1840
Also @Leemoonsoo review on this one would be helpful
---
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
Jeff Zhang created ZEPPELIN-1933:
Summary: Add queue configuration in pig interpreter
Key: ZEPPELIN-1933
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1933
Project: Zeppelin
Issue Type:
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1858
@corneadoug please 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 have this feature
enab
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1846
Thanks @Leemoonsoo Unit test is added. Please help review \cc @felixcheung
@jongyoul
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1804
I improved the template of html because I think `customHome` js controller
and `Home`js controller should be equally worked when `home` js controller is
changed.
So, I'll change from the temp
Tushar created ZEPPELIN-1934:
Summary: Building from source documentation
Key: ZEPPELIN-1934
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1934
Project: Zeppelin
Issue Type: Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1845
Sorry for late response @prabhjyotsingh, @jongyoul .
Let me review soon.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1761
@felixcheung Thanks for review.
Previously, we discussed about separating images in
[ZEPPEILN-1711](https://issues.apache.org/jira/browse/ZEPPELIN-1711)
IMO, having 1 image which
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1825
Thanks @minahlee!
Will merge if there are no more comments on 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1878
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
Renjith Kamath created ZEPPELIN-1935:
Summary: Add jceks stored password support for jdbc interpreter
Key: ZEPPELIN-1935
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1935
Project: Zeppelin
GitHub user r-kamath opened a pull request:
https://github.com/apache/zeppelin/pull/1881
ZEPPELIN-1935 Add jceks stored password support for jdbc interpreter
### What is this PR for?
Add support for jceks stored password instead of using password in clear
text. This security enh
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1871
Will merge it 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 feat
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1846
ZEPPELIN-1770. Restart only the client user's interpreter when restarting
interpreter setting
### What is this PR for?
This PR would only restart the trigger user's interpreter rather t
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1846
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
CI failure is not relevant.
```
AngularElem
- should provide onclick method *** FAILED ***
The code passed to eventually never returned normally. Attempted 1 times
over 602.10
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@Leemoonsoo @jongyoul @felixcheung @ejono Please help 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 n
GitHub user bitchelov opened a pull request:
https://github.com/apache/zeppelin/pull/1882
[MINOR] Small Selenium test fixes
### What is this PR for?
Remove unused imports;
Replace "Thread.sleep" (unstable solution);
Add "deleteTestNotebook()" method -> testWidth.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1846
CI failure is not relevant.
```
Failed tests:
NotebookTest.testAbortParagraphStatusOnInterpreterRestart:760
expected: but was:
AngularElem
- should provide onclick method
Github user Tagar commented on the issue:
https://github.com/apache/zeppelin/pull/1840
As far as credentials refresh are concerned, please see new comments in
[SPARK-19143](https://issues.apache.org/jira/browse/SPARK-19143).
Hope this helps.
---
If your project is set up for it,
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1852
---
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/1860
---
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/1874
---
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/1789
---
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/1882
@bitchelov Thanks for making selenium test more stable!
LGTM and merge to master if no more discussions.
---
If your project is set up for it, you can reply to this email and have your
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1875
LGTM and merge to master if 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 have thi
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1878
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
Thanks Mina taking care of it. Sounds good to me.
One thing is, shell we try have docker image in this release?
Following issues tracks the the job
https://issues.apache.org/jira/browse/ZEPPELIN-1711
https://issues.apache.org/jira/browse/INFRA-12781
I don't think it need to be a blocker for 0.7.0
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1863
---
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/1871
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
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1880
wait, in travis, I don't think we want to replace 2.0.1 with 2.1.0?
generally for new Spark minor version we create a new test profile for it so we
would test all supported Spark versions (yes
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1880
LGTM. Sorry I didn't spot the version number earlier.
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1880
Tested and it works well.
Shell we add `spark-2.1` profile
[here](https://github.com/apache/zeppelin/blob/master/docs/install/build.md#build-profiles)
and
[here](https://github.com/a
Thanks Mina Lee.
On Tue, Jan 10, 2017 at 11:10 AM moon soo Lee wrote:
> Thanks Mina taking care of it. Sounds good to me.
>
> One thing is, shell we try have docker image in this release?
> Following issues tracks the the job
> https://issues.apache.org/jira/browse/ZEPPELIN-1711
> https://issues
Github user ejono commented on the issue:
https://github.com/apache/zeppelin/pull/1880
I built with "./dev/change_scala_version.sh 2.11; mvn clean package
-Pscala-2.11 -Pspark-2.1" and see the same test failure that Travis saw for the
new 2.1.0 build (https://travis-ci.org/apache/zepp
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1845
@prabhjyotsingh Thank you for great improvement!
I tested this with mysql and all features are working well except one.
In following case, database connection would be closed and user shou
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1876
@Leemoonsoo
I'v just rebased
---
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
ena
Github user placeybordeaux commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Not sure why the CI hasn't kicked in, but I tried compiling this on my
local and I am getting some test failures:
```
Results :
Failed tests:
NotebookTest.testAbo
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1868
@Leemoonsoo now `python.conda` support other `env *` commands :)
https://cloud.githubusercontent.com/assets/4968473/21827211/0130cd40-d7ce-11e6-8925-4757c6a2d6fc.png";>
---
If your projec
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1831
@zjffdu I'v just fixed not to extend PYTHONPATH using submitted packages
only in yarn-client
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Thank you Mina for taking care of this tasks!
On Wed, 11 Jan 2017 at 5:14 AM Vinay Shukla wrote:
> Thanks Mina Lee.
>
>
>
> On Tue, Jan 10, 2017 at 11:10 AM moon soo Lee wrote:
>
>
>
> > Thanks Mina taking care of it. Sounds good to me.
>
> >
>
> > One thing is, shell we try have docker image i
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1878
Merge if there are no more comments on 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
Thank you Mina for taking care of this.
On 11 Jan 2017 7:23 a.m., "Anthony Corbacho"
wrote:
> Thank you Mina for taking care of this tasks!
>
> On Wed, 11 Jan 2017 at 5:14 AM Vinay Shukla wrote:
>
> > Thanks Mina Lee.
> >
> >
> >
> > On Tue, Jan 10, 2017 at 11:10 AM moon soo Lee wrote:
> >
> >
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
Agree with @Leemoonsoo
---
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 wi
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1831
Thanks, @1ambda Do you mind to create a spark ticket as well ? The behavior
inconsistency between different modes seems an issue of spark, we need to
clarify it with spark community.
---
If your
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1825
---
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/1878
---
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
Moon, thank you for reminding.
I think it would be beneficial to community to provide public docker image.
I will definitely take it into account as infra part already has been taken
cared and Dockerfile is almost ready.
On Wed, Jan 11, 2017 at 4:10 AM moon soo Lee wrote:
> Thanks Mina taking ca
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@ejono I have a following commit to fix the test failure. Could you retry
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
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1871
Tested and it works well :+1:
---
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 an
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@Leemoonsoo @AhyoungRyu build.md is updated, but I didn't touch
`create_release.sh`, I will left it to @minahlee to decide.
---
If your project is set up for it, you can reply to this email and
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1876
---
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/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 is e
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1880
We always have been deliver release with the most recent minor spark
release, so I think it's better to update release profile to use 2.1.
@zjffdu It would be appreciated if you can update spa
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@minahlee Thanks for help. I have updated these release script.
---
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1845
That logic is something that I have not changed, and I think we can take
care/discuss it in a different PR.
This is what I've tried on the existing functionality.
![test-mysql]
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@cloverhearts I'd like to test this. could you resolve some conflict file?
---
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 p
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1842
It's very cool feature!
Tested and It works very nice like below.
**[Test result]**
When I clicked `Enable` button for using `zeppelin_horizontalbar`
visualization package.
!
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1842
And when I test npm online package for using bubble chart, It works very
well!
![z1842_b](https://cloud.githubusercontent.com/assets/8110458/21835960/94d69aee-d804-11e6-8b4f-8b51c0bf72cc.gif)
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1840
I just pushed changes to keep compatibility using
`ZEPPELIN_IMPERSONATE_SPARK_PROXY_USER` env. variable that will disable usage
of `--proxy-user` option. after
[SPARK-19143](https://issue
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@soralee
Sure :)
rebase is done.
---
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 th
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1845
@prabhjyotsingh Yea I know you didn't change it. That's because it doesn't
have state before.
I agree that we can take care/discuss it in a different PR.
Thanks.
---
If your proje
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1831
**For reviewers**
Fixed to use `spark.jars` instead of `classpath`.
- classpath doesn't include submitted jars at this moment (i could get 7
days ago, but not now)
- it enable
Thanks Mina! 👍
2017-01-10 18:23 GMT-08:00 Mina Lee :
> Moon, thank you for reminding.
> I think it would be beneficial to community to provide public docker image.
> I will definitely take it into account as infra part already has been taken
> cared and Dockerfile is almost ready.
>
> On Wed, Jan
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1880
great, we are good to go then, yeah?
@minahlee btw, should we update the hadoop-2.4 in the release script - we
are testing against 2.6 in CI and almost everyone out there should be run
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1875
hmm, it doesn't break RAT tests for all these binaries? :)
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1881
this is important - is there any backward compatibility concerns with
changing the password storage?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1846
Tests LGTM. Thanks!
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1871
---
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 r-kamath commented on the issue:
https://github.com/apache/zeppelin/pull/1881
@felixcheung JCEKS is used only when the password field is empty. When
password and JCEKS properties are empty the connection attempt is made without
password.
---
If your project is set up for
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@felixcheung Let me update release script in another PR. @zjffdu do you
mind to resolve conflict?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1834
@soralee Thanks for the update and kind reminder! Let me test 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 p
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1881
@r-kamath IMO this shouldn't be called "backward compatibility" as these
are two ways in which password can provided to interpreter.
Tested locally, works as expected, LGTM!
---
I
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1880
@minahlee PR is rebased.
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
ping @AhyoungRyu , please check the updated screenshot
---
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1883
Bump up version to 0.8.0-SNAPSHOT
### What is this PR for?
Bump up version to 0.8.0-SNAPSHOT
You can merge this pull request into a Git repository by running:
$ git pull https://github.
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1880
LGTM. Merge to master and branch-0.7 if there is 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Sorry for my late response. It looks nice! Except one minor suggestion,
LGTM ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Khalid Huseynov created ZEPPELIN-1936:
-
Summary: User Impersonation setting isn't propagated when creating
new interpreter
Key: ZEPPELIN-1936
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1936
I would like to inform you that branch-0.7 has cut.
You can check it through apache git repository URL [1] but it seems like it
takes some time for Github mirror.
PMCs/Commiters, please note that you need to merge bug fix both to master
and branch-0.7
if it needs to be included in 0.7.x release.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Thanks @AhyoungRyu , I rename the note, please check the updated screenshot
in PR description.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1758
@AlexanderShoshin Checked again and it looks nice. Except the minor
suggestion, LGTM. Thanks for your effort and the awesome tutorial note!
---
If your project is set up for it, you can repl
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/1884
[ZEPPELIN-1936] Fix user impersonation setting propagation
### What is this PR for?
This is to fix bug of creating interpreter with impersonation enabled
### What type of
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1834
You mean if we
open the folder in the **_navbar_** (e.g. Zeppelin Tutorial)-> type some
text in the search bar -> remove all text -> the opened folder is closed
But this isn'
Jeff Zhang created ZEPPELIN-1937:
Summary: Improve the logic of splitting pig statements
Key: ZEPPELIN-1937
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1937
Project: Zeppelin
Issue Ty
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Thanks! Merge into master if there are 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
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1881
Got 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 w
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1834
@AhyoungRyu Yes, you're right.
I mean, If I change **navbar** folder option, it works together both the
filter in the **home** and **navbar**. So, I did not include this part because
I think
Github user soralee closed the pull request at:
https://github.com/apache/zeppelin/pull/1834
---
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 soralee reopened a pull request:
https://github.com/apache/zeppelin/pull/1834
[ZEPPELIN-1864] Improvement to show folder and note after searching note
### What is this PR for?
This PR is for improvement to show folder and note after using the filter.
And I found some
94 matches
Mail list logo