GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1404
ZEPPELIN-1411. UDF with pyspark not working - object has no attribute
'parseDataType'
### What is this PR for?
The root cause is that SQLContext's signature changes in spark 2.0.
Spark
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1404
---
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 khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1408
@kavinkumarks this looks better and tested locally, works as expected. few
things:
* would be nice to update screenshot above
* CI build fails, could you take a look into that or re
Github user kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1393
Thanks @Leemoonsoo for reviewing this! Could you please merge this?
- Kavin
MailTo: kavin.ku...@imaginea.com
---
If your project is set up for it, you can reply to this email and
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@Leemoonsoo issues are created under
[ZEPPELIN-1437](https://issues.apache.org/jira/browse/ZEPPELIN-1437) and
[ZEPPELIN-1438](https://issues.apache.org/jira/browse/ZEPPELIN-1438)
correspon
Khalid Huseynov created ZEPPELIN-1438:
-
Summary: Per user note list broadcast on note permission change
Key: ZEPPELIN-1438
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1438
Project: Zeppelin
Khalid Huseynov created ZEPPELIN-1437:
-
Summary: Note list broadcast is not aware of user session
Key: ZEPPELIN-1437
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1437
Project: Zeppelin
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1423
Thanks for the explanation.
LGTM and merge 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. I
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1411
LGTM and merge 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 have this
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1423
@Leemoonsoo The `rpkg.zip` is for the case that user have R code inside of
jars specified through --jars. The reason why I didn't include it here is that
I have to duplicate lots of code from spa
Github user rfilmyer commented on the issue:
https://github.com/apache/zeppelin/pull/1416
Alright. I'm probably going to make some more changes on this PR. (Probably
2 or 3 more pages).
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1418
Merge if there're 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 this feat
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1416
Thanks @rfilmyer for the great contribution!
Like @AhyoungRyu explained, you can either add more commits here or create
new PR depending on further contribution you want to make.
LG
Deenar Toraskar created ZEPPELIN-1436:
-
Summary: Python Interpreter does not work on Windows
Key: ZEPPELIN-1436
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1436
Project: Zeppelin
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1426
I think displaying result with additional information is quick and simple
approach. However, i think there're some downsides like
- can not leverage output streaming
- when user out
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1384
Thanks @1ambda. Looks great to me.
Merge if there're 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.
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1423
@zjffdu Do you think
[rpkg.zip](https://github.com/apache/spark/blob/v1.6.2/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala#L394)
is the something also need to be taken care?
--
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1425
Thanks @IshmeetKaur. I think i'll try to merge #1422 first because #1422
arrived earlier.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1422
Could you try indent code with 2 spaces like other part of codes do?
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@mwkang @khalidhuseynov Could you handle or at least create issues for
expected problems after merge this PR? I can at least think following two
issues,
- Note list broadcasting messag
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1393
Thanks @kavinkumarks for the contribution.
This changes not only resolving race condition described in ZEPPELIN-1142
but also changes user experience. i.e.
Before, owner is not defi
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1424
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1413
---
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/1404
Thanks @zjffdu. I think second ci test profile failure looks relevant.
```
ests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 120.231 sec
<<< FAILURE! - in org.apache.ze
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@astroshim @corneadoug, Shell we have some unittests for `public void
restart(String settingId, String noteId)` before we merge? Restarting only a
interpreter instance/process for a note is new
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1427
[ZEPPELIN-1435] Move zeppelin-server-*.jar under lib in distribution binary
package
### What is this PR for?
Move zeppelin-server-*.jar package under `lib` directory like other
zeppelin mod
Mina Lee created ZEPPELIN-1435:
--
Summary: Move zeppelin-server-*.jar under lib directory in
distribution package
Key: ZEPPELIN-1435
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1435
Project: Zeppe
Github user IshmeetKaur closed the pull request at:
https://github.com/apache/zeppelin/pull/1425
---
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
GitHub user IshmeetKaur reopened a pull request:
https://github.com/apache/zeppelin/pull/1425
Improving Matplotlib Integration with Zeppelin
### What is this PR for?
This PR improves the matplotlib integration with zeppelin.
### What type of PR is it?
Improvement
David Tzoor created ZEPPELIN-1434:
-
Summary: Long numbers rounding in Zeppelin-Web table
Key: ZEPPELIN-1434
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1434
Project: Zeppelin
Issue Ty
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1398
@zjffdu It's okay :) 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
enable
sam created ZEPPELIN-1433:
-
Summary: What dependency to add to project (sbt or maven) get hold
of ZeppelinContext?
Key: ZEPPELIN-1433
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1433
Project: Zeppelin
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1398
[MINOR] Doc fix for spark interpreter
### What is this PR for?
Several changes on doc of spark interpreter.
* %spark, %sql, %pyspark only works when spark is the default interpreter
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1398
Sorry @AhyoungRyu I didn't notice you PR is based on my branch and thought
you will merge to master directly.
---
If your project is set up for it, you can reply to this email and have your
repl
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1398
@zjffdu Did you close this PR on purpose?
---
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 zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1398
---
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/1404
@Leemoonsoo , I updated the unit test. And also made a little change
`AbstractTestRestApi`, the standalone way doesn't work for me. So I allow user
to export `SPAKP_MASTER` to run it in other modes
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1412
---
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 corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1302
Merging if there is 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 this featur
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1358
@Leemoonsoo 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 this feature
enabled and wishes so
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1426
Here is before and after of this PR that I observe, and I see these 2
changes
- now it does not stream output, i.e. it waits for the entire paragraph
execution to be finished, and
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1424
Looks good to me.
Will merge to master if there is no further 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 you
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1426
@prabhjyotsingh @Leemoonsoo @felixcheung @bzz @minahlee Please help review.
I changed the livy output display style from plain text to angular in order
to display the link. And for now it i
43 matches
Mail list logo