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
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
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
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
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
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
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
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
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
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/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/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
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
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.
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)
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
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
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:
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
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
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
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:
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
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
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
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
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1330
I think it doesn't solve all problems, but definitely in the roadmap. 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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1426
ZEPPELIN-1430. Display appId and webui link in LivyInterpreter's output
### What is this PR for?
For now, it is hard to figure out what the yarn application of the livy
session represent, it
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1330
Thank you @mwkang @khalidhuseynov !
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 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
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:
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
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
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
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
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,
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
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
38 matches
Mail list logo