Github user purechoc commented on the issue:
https://github.com/apache/zeppelin/pull/1587
can't do that,
%livy.pyspark support python2 only.
---
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/1596
@Leemoonsoo Let me know if this is correct, or I may miss something.
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1596
ZEPPELIN-1616. Interpreter open happens in jobRun
### What is this PR for?
Not sure why we put interpreter open in jobRun, that means the thrift
service method open is never used. I think open
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1587
Hmm, can't we do that with the existing `%livy.pyspark` interpreter?
---
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
Jeff Zhang created ZEPPELIN-1616:
Summary: Interpreter open happens in jobRun
Key: ZEPPELIN-1616
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1616
Project: Zeppelin
Issue Type: Bug
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1594
Thanks for the review @bzz, @felixcheung! I'v just fixed :)
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@anthonycorbacho @khalidhuseynov Thanks for the comment! I agree with you
guys.
This is a breaking change, so yeah we must be careful. So, if we proceed
this, new API URL's prefix will be
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1591
travis is super slow today
Amerika is working to hard today :)
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1591
Looks great to me, thank you for fix and prompt response!
Let's merge after CI is green, if there is no further discussions.
---
If your project is set up for it, you can reply to this emai
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1591
@anthonycorbacho Thanks for bringing back async nature for note saves, LGTM!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@tae-jun thanks for summarizing and giving feedback on REST API. Although
we can't address all at once, but I think it deserves an issue with above
content and feedback logged in there
--
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1593
@jongyoul I fix the combination issue and also did some refactoring.
* rename interpreterInstanceKey to sessionKey
---
If your project is set up for it, you can reply to this email and have yo
GitHub user anthonycorbacho opened a pull request:
https://github.com/apache/zeppelin/pull/1595
[ZEPPELIN-1615] - Zeppelin should be able to run without Shiro
### What is this PR for?
Right now, Zeppelin use Shiro by default even if you dont need it.
(It will use shiro.ini.te
Anthony Corbacho created ZEPPELIN-1615:
--
Summary: Zeppelin should be able to run without Shiro
Key: ZEPPELIN-1615
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1615
Project: Zeppelin
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1594
This is awesome, thanks!
+1 on @bzz 's 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 hav
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@tae-jun This is a great sum of the status of the rest api in zeppelin, and
i think we all agree that currently zeppelin doenst follow rest standard (just
by looking at the endpoint + resp
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1594
Websequence are awesome, thank you @1ambda for contributing this
simplification of MD interpreter.
Looks good to me, modulo few minor issues, commented above.
As soon as CI is green
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee Oh, in that context, it makes sense. LGTM.
But, after looking all around Notebook REST API, it works well now but it
doesn't seem RESTful. I summarized current list of REST API:
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1593
Thanks for creating this issue, this wasn't working for me either.
---
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 pr
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1471
So cool guys, sorry, I have totally missed out on this - it looks great and
CI is green now!
Looks like every feedback has been addressed by now?
Quick question on test and CI - it s
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/1594
[ZEPPELIN-1614] Remove markdown4j dep
### What is this PR for?
Remove unmanaged, old library markdown4j dep which exists just for
Websequence and YUML plugins.
(related to https://gi
Hoon Park created ZEPPELIN-1614:
---
Summary: Remove markdown4j dependency by adding plugins to pegdown
Key: ZEPPELIN-1614
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1614
Project: Zeppelin
Github user agoodm commented on the issue:
https://github.com/apache/zeppelin/pull/1534
@felixcheung
In that case, I think that should be fine. Will do shortly.
@Leemoonsoo
The latest commit should fix the issue I just mentioned. Two other
important things to mention
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1593
\cc @jongyoul @Leemoonsoo 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 not have this fe
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1593
ZEPPELIN-1613. PerUser scoped mode doesn't work
### What is this PR for?
For perUser scoped mode, difference users still use the same interpreter
instance which is incorrect. This is due to th
Jeff Zhang created ZEPPELIN-1613:
Summary: PerUser scoped mode doesn't work
Key: ZEPPELIN-1613
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1613
Project: Zeppelin
Issue Type: Bug
A
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1534
Great.
I think it will be better to have interpreter/lib - it might be easier to
set ACL on a common interpreter root in the event there are user downloaded
interpreters for examples.
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1592
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1590
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1589
ð for docs in same PR.
On CI, I think that's relativly minor and if that's too complicated to
configure - we can can opt out for manual test run+instruction in docs.
Double-checki
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
rebuild CI
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user/pas
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 purechoc commented on the issue:
https://github.com/apache/zeppelin/pull/1587
first, you need to install livy and setup python3 environment.
second, set config. "spark-defaults.conf"
spark.yarn.appMasterEnv.PYSPARK3_DRIVER_PYTHON usr/bin/python3
and then, ju
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1587
hi - thanks for the contribution. How is this going to start python3? could
you elaborate?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1578
Looks great to me.
Nice side-effects is that CI should become faster now.
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1590
Got it! Thank you for kind explanation.
CI failure looks not relevant for the change set.
Looks great to me, merging to master if there is no further discussion.
---
If your project
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1572
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/320
This looks to be an unique and important feature to have, will be great to
have this in Zeppelin
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user agoodm commented on the issue:
https://github.com/apache/zeppelin/pull/1534
@felixcheung
I updated the documentation and also added the example gif shown in this
PR. In regards to getting python dependencies installed for our CI tests, I
would be glad to help with tha
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1584
@felixcheung @prabhjyotsingh Thanks for the review!
Will merge into `gh-pages` branch if there are no more discussions.
---
If your project is set up for it, you can reply to this email a
Github user mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@khalidhuseynov Thank you for your reviews. For the point # 1, I will work
on enhancing this in next pull request. And for 2 and 3, the docker image could
be run in the daemon mode and I added i
Github user mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@astroshim R and Python are installed and I think that the tutorials should
be run without errors.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1583
---
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/1579
---
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/1573
---
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/1535
---
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/1535
Thanks @kavinkumarks for resolving conflict. CI failure looks irrelevant.
I'm merging it into master.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1590
@bzz that function `loadAllNotes` is called only from constructor and when
loading from constructor you're either in anonymous mode, or in shiro and
loading all notes by anonymous (public).
Github user hkropp commented on the issue:
https://github.com/apache/zeppelin/pull/1589
Provided documentation as well.
---
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 hkropp commented on the issue:
https://github.com/apache/zeppelin/pull/1589
Thanks for your feedback @bzz
I applied the changes required for 1. and 2.
3. Concerning Travis:
It would need to be a username and password local to the machine the test
is b
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1592
[ZEPPELIN-1029] Add rest api for paragraph config update
### What is this PR for?
This PR adds new end point for updating paragraph configuration
### What type of PR is it?
Impro
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@tae-jun the reason I use `PUT` is that `clearAllParagraphOutput` method is
not really deleting `result` field from paragraph, but updating `result` field
to empty.
I was considering using `D
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee I have a question about REST API style. Is there any purpose that
you used `PUT` method?
In my opinion, `DELETE` method is more proper since it does not `PUT`
`clear` under a `n
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1589
Thank you for contribution!
There are few things need to be taken care here:
1. make sure the code adhere [project
styleguide](https://zeppelin.apache.org/contribution/contributions.html#
GitHub user anthonycorbacho opened a pull request:
https://github.com/apache/zeppelin/pull/1591
Fix latency on notebook save in ZeppelinHubNotebookRepo
### What is this PR for?
Fix latency issue when user save a notebook while using zeppelinhub
notebook storage.
### Wha
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1564
---
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/1565
@khalidhuseynov Thank you for quick response!
Merging 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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1590
Just wonder if that means that notes are always load as `anonymous`
irregarding Zeppelin mode, because we do not have such permissions yet?
Curious, as I would expect that we would load all no
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee looks much better now! thanks for addressing the issue and 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1581
@agura Thanks for your quick response. Except for
`../manual/interpreters.html#what-is-interpreter-setting` link, LGTM.
---
If your project is set up for it, you can reply to this email and h
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1578
ð
---
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 fe
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/1590
[ZEPPELIN-1612] Fix NPE when initializing Notebook
### What is this PR for?
Sometimes Zeppelin wasn't able to start because of empty subject when
initializing Notebook class, more deta
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1578
@jongyoul [This
part](https://github.com/jongyoul/zeppelin/blob/acf514f05eaeedd2f8de0aa4f4002aa258b5aadd/testing/downloadSpark.sh#L30)
also needs to be updated accordingly I guess.
---
If y
Hi devs and users,
Spark 1.1.x to 1.3.x is no longer supported from Zeppelin 0.7.0. Those
versions are also moved to archive.apache.org.
Please check the PR below and leave a comment if you have any issue for it:
https://github.com/apache/zeppelin/pull/1578
Regards,
Jongyoul Lee
--
이종열, Jongy
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1578
@minahlee I've fixed what you mentioned.
Merging 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 w
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
please 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
enabled and wishes so, or
Khalid Huseynov created ZEPPELIN-1612:
-
Summary: NPE when starting Zeppelin because of non-initialized
subject
Key: ZEPPELIN-1612
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1612
Project:
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1535
Resolved the conflicts and the build ran but there are failures related to
spark for 2 consecutive times.Refer below for details:
`Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Ti
GitHub user hkropp opened a pull request:
https://github.com/apache/zeppelin/pull/1589
[Zeppelin-1611] - Support PAM (System User) Authentication
### What is this PR for?
This PR adds
[PAM](https://en.wikipedia.org/wiki/Pluggable_authentication_module)
authentication support b
Github user kavinkumarks closed the pull request at:
https://github.com/apache/zeppelin/pull/1535
---
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 kavinkumarks reopened a pull request:
https://github.com/apache/zeppelin/pull/1535
[Zeppelin-53] Broken newly created Notebook if there is no permissions to
mkdir on disk
### What is this PR for?
Error message should be shown to the user when there is permission
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1535
Reopening to trigger the CI build.
---
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
e
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1586
---
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
Henning Kropp created ZEPPELIN-1611:
---
Summary: Support PAM (System User) Authentication
Key: ZEPPELIN-1611
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1611
Project: Zeppelin
Issue
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1586
Thank you @khalidhuseynov for prompt 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
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1586
Tested and 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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1586
Looks great to me, thank you for updated integration tests @cloverhearts !
CI fails on 2 profiles
```
Results :
Tests in error:
InterpreterRestApiTest.init:57->Abs
GitHub user anthonycorbacho opened a pull request:
https://github.com/apache/zeppelin/pull/1588
[ZEPPELIN-1610] [WIP] - Add notebook watcher
### What is this PR for?
Add a Simple way to switch a websocket connection to a new state; watcher.
A websocket watcher is a special co
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@khalidhuseynov I updated rest api to do privilege check. Could you 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. I
Anthony Corbacho created ZEPPELIN-1610:
--
Summary: Be able to monitor Zeppelin websocket even
Key: ZEPPELIN-1610
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1610
Project: Zeppelin
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1583
Good catch!
---
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 anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1583
merging if 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 feature
e
Github user purechoc commented on the issue:
https://github.com/apache/zeppelin/pull/1447
LGTM. it's working nicely in my environment. (zeppelin 0.7 + livy 2.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 pro
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1583
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 purechoc opened a pull request:
https://github.com/apache/zeppelin/pull/1587
using pyspark(python3) with livy interperter
### What is this PR for?
using pyspark(python3) with livy interperter
### What type of PR is it?
Improvement
### Todos
hyunwoo cho created ZEPPELIN-1609:
-
Summary: using python3 with livy interperter
Key: ZEPPELIN-1609
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1609
Project: Zeppelin
Issue Type: Impr
87 matches
Mail list logo