Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3240
> Awesome! that is a great news. Just out of curiosity, does the Zeppelin
server in KB must be the one which has acess to the Spark_Home or the Spark
interpreter which is now a pod? (Just
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3240
> One question, there is no point using this if I have a Spark clster on
YARN (Cloudera/Hortonworks) right? The way I see it this is focusing on having
The Spark coming with Zeppelin now
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3240
[ZEPPELIN-3840] Zeppelin on Kubernetes
### What type of PR is it?
This PR adds ability to run Zeppelin on Kubernetes. It aims
- Zero configuration to start Zeppelin on
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3240
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3240
[ZEPPELIN-3840] Zeppelin on Kubernetes
### What type of PR is it?
This PR adds ability to run Zeppelin on Kubernetes.
Key features are
- Provides zeppelin-server.yaml
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3231
Merge to master, if no further comments.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3230
Thanks @zjffdu, @felixcheung for review. Merge to master and branch-0.8.
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3231
[MINOR] Let jenkins print some build summary on failure
### What is this PR for?
Currently actual CI build occurs in individual contributor's travis-ci
account, and apache Jenkins
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3231
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3232
[ZEPPELIN-3838] Input function example code incorrect on "What is Dynamic
Form" page
### What is this PR for?
Input function example screenshot is incorrect on "What
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3229
Thanks @egorklimov for the improvement and attaching the benchmark result
LGTM
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3231
Let jenkins print some build summary on failure
### What is this PR for?
Currently actual CI build occurs in individual contributor's travis-ci
account, and apache Jenkins poll the r
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3230
[ZEPPELIN-3847] Duplicate results in notebooks due to WS interruption
### What is this PR for?
Duplicate results in notebooks after ws reconnect.
The problem occurs via duplicated
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3226
Thanks @Savalek for the contribution. I have a few questions.
1. What happens if a browser opens running notebook? Is it correct that
`SEQUENTIAL_RUN_STATUS` message will not be
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3225
This PR has been merged to branch-0.8. But somehow @asfgit didn't close the
PR. @jolks Do you mind close this PR manually?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3224
Thanks @jolks for the contribution.
LGTM and merge to master
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3217
Thanks @felixcheung for review. merge to master.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3220
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3219
Thanks @felixcheung for review. Merge to master.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3218
Thanks @felixcheung @zjffdu for review. Merge to master.
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3219
[ZEPPELIN-3850] Fix run all paragraph behavior
### What is this PR for?
"Run all paragraphs" button runs only the first paragraph of the note.
This PR fixes t
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3218
[ZEPPELIN-3849] display note name correctly.
### What is this PR for?
Master branch displays note id instead of note name.
I think it's side effect after change
https://githu
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3207
@88sanjay Cool! LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3216
Thanks @NicolasRouquette for the contribution.
Looks like CI test is failing at
```
Results :
Failed tests:
SparkRInterpreterTest.testSparkRInterpreter:88
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3212
Merge to master
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3214
Merge to master
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3217
[ZEPPELIN-3848] zeppelin-web dev mode does not work
### What is this PR for?
zeppelin-web development mode is broken. `npm run dev` throws error
```
Error: Cannot find module
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3214
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3212
[ZEPPELIN-3837] update how_to_build.md
### What is this PR for?
how_to_build.md includes some build profiles that has removed/changed.
This PR makes how_to_build.md up to dated
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3212
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3214
[ZEPPELIN-3836] [FOLLOWUP] Update link in README.md
### What is this PR for?
After https://github.com/apache/zeppelin/pull/3211, some link in README.md
need to be updated
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3214
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3214
[ZEPPELIN-3836] [FOLLOWUP] Update link in README.md
### What is this PR for?
After https://github.com/apache/zeppelin/pull/3211, some link in README.md
need to be updated
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3207
Thanks @88sanjay for the contribution. Few problem on CI test has been
fixed recently. Do you mind rebase your branch and see if this PR passes CI
test just in case?
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3212
[ZEPPELIN-3837] update how_to_build.md
### What is this PR for?
how_to_build.md includes some build profiles that has removed/changed.
This PR makes how_to_build.md up to dated
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3212
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3214
[ZEPPELIN-3836] [FOLLOWUP] Update link in README.md
### What is this PR for?
After https://github.com/apache/zeppelin/pull/3211, some link in README.md
need to be updated
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3211
Thanks @felixcheung. Merging and deploy this change.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3213
Thanks @felixcheung review.
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3213
[ZEPPELIN-3839] Replace | to , in the filename
### What is this PR for?
A tutorial notebook filename, which includes `|`, is not compatible with
windows filesystem.
This PR replace
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/3212
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/3212
[ZEPPELIN-3837] update how_to_build.md
### What is this PR for?
how_to_build.md includes some build profiles that has removed/changed.
This PR makes how_to_build.md up to dated
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3212
[ZEPPELIN-3837] update how_to_build.md
### What is this PR for?
how_to_build.md includes some build profiles that has removed/changed.
This PR makes how_to_build.md up to dated
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3211
[ZEPPELIN-3836] update website url rewrite rule for 0.8.0 release
### What is this PR for?
`https://zeppelin.apache.org/docs/latest` supposed to point latest release
version documentation
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3210
Thanks @zjffdu for a review.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3210
@zjffdu @felixcheung please review.
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3210
[ZEPPELIN-3835] Update website for 0.8.0 release
### What is this PR for?
0.8.0 has released in last June, but what's new section in website still
have information of 0.7.x.
Th
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3061
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3052
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3049
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3050
[CI
green](https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/399684108).
Merge this to master.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3051
Thanks @Byeol for contribution!
Merge this as a hotfix.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3049
Thanks @oxygen311 for the contribution. Could you resolve conflict and see
if it passes CI?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3050
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3033
LGTM
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/3036
[ZEPPELIN-3554] Flaky test - InterpreterContextTest.testThreadLocal
### What is this PR for?
Fix flaky test. See
[ZEPL-3554](https://issues.apache.org/jira/browse/ZEPPELIN-3554
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2991
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2986
I'd like to separate Bi-directional RPC and Bi-directional network
connection.
Of course Bi directional RPC of course gives lots of benefits. But
Bi-directional network conne
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2986
I was thinking the cases like a (or more) firewalls, NATs between Server
and Interpreters.
And accepting range of ports on each side is more difficult to configure
than accepting a single
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2986
Thanks @zjffdu for the improvement.
Overall looks good to me. I'd like to see CI goes green before merge while
it change code a lot.
One question. Let's say in
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/2956
[DOC] Fix broken link to writing visualizations doc
### What is this PR for?
This PR fixes broken link to doc. Need to be merged branch-0.8.
### What type of PR is it?
Bug Fix
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2841
Thanks @Savalek for great improvement. LGTM and merge to master and
branch-0.8 if no further comment.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2911
Thanks @felixalbani. merging it.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2867
Thanks @felixalbani for contribution. I'm merging it to master and
branch-0.8 if no further comment.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2867
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2760
Took a look changes and it looks good to me
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2842
This pullrequest is merged to `branch-0.8`. But asfgit couldn't
automatically close it. @weand Could you close it manually?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2842
Thanks for the fix. Let me merge this to branch-0.8.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2763
Thank you. Marked ZEPPELIN-3194 as resolved.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2829
Thanks @zjffdu @felixcheung for review. Merge to master and branch-0.8
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2828
Thanks @masyan @zjffdu @felixcheung for review. Merge to master and
branch-0.8
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2763
@masyan Do you have your JIRA id?
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/2829
[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python
interpreter
### What is this PR for?
```
%python
import time
print("Hello")
time
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/2829
---
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/2829
---
GitHub user Leemoonsoo reopened a pull request:
https://github.com/apache/zeppelin/pull/2829
[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python
interpreter
### What is this PR for?
```
%python
import time
print("Hello")
time
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2827
LGTM
---
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/2829
[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python
interpreter
### What is this PR for?
```
%python
import time
print("Hello")
time
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/2828
[ZEPPELIN-3283] Remove redundant method
InterpreterOutput.getInterpreterResultMessages()
### What is this PR for?
Remove Redundant method `InterpreterOutput.getInterpreterResultMessages
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2763
Thanks @masyan for great contribution. Looks good to me.
I'll merge this to master and branch-0.8 if no further comments.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2817
Interesting feature, thanks @SleepyThread for the contribution.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2811
[Multiple builds](https://travis-ci.org/wegtam/zeppelin/builds/344196467)
fails with
```
artifacts could not be resolved:
org.apache.flink:flink-clients_2.10:jar:1.4.1
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2735
Tested and it works well.
LGTM and merge to master and branch-0.8.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2771
I ran CI test of this branch
https://travis-ci.org/Leemoonsoo/zeppelin/builds/343475153 in my travis
account. And build went success. I think previous error was flaky test
unrelated to this
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2771
Okay, let me take a look and see if errors are related to this change or
not.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2763
I'd like to merge this PR to master if we have no further reviews. @masyan
Is this PR closed on purpose?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2770
LGTM! Merge to master and branch-0.8 if no further discussions.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2771
Thanks @sancyx for the contribution.
It looks good, but i'd like to see if it passes CI.
Could you try setup CI and trigger build?
```
Looks like travis-ci i
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2770
Tested and fix works well. But it's getting build failure with following
error.
```
[INFO]
[INFO] /zeppelin/zeppelin-web/src/app/search/result-list.controller.js
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2776
@srikgn Could you try include commits related to this contribution only?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2775
Thanks for the contribution.
But this pullrequests includes unrelated commits to this contribution.
@srikgn Could you try include commits related to this contribution only?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2763
Cool! Looks great to me.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2774
Merge to master and branch-0.8
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2774
Thanks @Savalek for the improvement. LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2754
Merge this to master and branch-0.8
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2709
Thanks @zjffdu for explain. Looks good to me.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2754
Thanks for the fix. LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2709
in the binary package, how spark/scala-2.11 and spark/scala-2.10 modules
will be packaged?
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2745
Merge to master if no further comment
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2745
Thanks @necosta for the fix. LGTM
---
1 - 100 of 1674 matches
Mail list logo