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? (J
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
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
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 poll
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 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 result
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
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 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://github.com
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
@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.
This PR
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 connection
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 the future
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/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
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
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
[INFO
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
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2711
LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2698
@tinkoff-dwh Looks good to me!
Merge to master if no further comment.
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2687
Thanks @kjmrknsn. Merge to master if no further review!
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2696
Thanks @namanmishra91. Looks great to me!
Merge to master if no further comment.
---
1 - 100 of 1451 matches
Mail list logo