GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1108
[ZEPPELIN-1081] Fixed not to include spark.r setting while not using
-Psparkr
### What is this PR for?
Avoid setting `spark.r` without `-Psparkr`. This enables to use `-Pr`
correctly
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1108
Ready to review. This is related to #1077
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1108
I'll merge it into master and branch-0.6
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/957
@bzz I'll continue to work with it after interpreter impersonation.
---
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 jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/957
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1087
The failure is flaky and irrelevant but we need to ensure all other tests
passed. Could you please re-trigger CI?
---
If your project is set up for it, you can reply to this email and have your
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1077
@rerngvit Could you please try to build with `-Pr` after rebase it from
current master?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1087
@fvaleri Got it. 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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1102
Merging 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 well. If your project does not have this fe
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1112
[MINOR] Enable launching in debug mode
### What is this PR for?
Running ZeppelinServer.main normally. It will enable user to debug
ZeppelinServer easily.
### What type of PR is it
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1112
It works in Intellij. Could anyone check if it works on Eclipse?
---
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 jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1112
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1112
[MINOR] Enable launching in debug mode
### What is this PR for?
Running ZeppelinServer.main normally. It will enable user to debug
ZeppelinServer easily.
### What type of PR is
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1112
BTW, it has a problem to run `mvn verify`. I'll fix it
---
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 doe
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/
I'm testing it. Just wait.
---
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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/
@prabhjyotsingh After run all paragraphs, note adds new paragraph at the
end of last paragraph. And then, if the last paragraph is empty, note don't
create paragraph anymore. Is it norm
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/
@prabhjyotsingh I only said that case of "running all paragraph". I think
"running all paragraphs" is not interactive mode. But in a normal case, I like
to add empty par
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/
@prabhjyotsingh You're right for the before behaviour. I thank you for
taking care of it. In case of your PR, I've tested and worked well. I hope you
can deal with my suggestion.
-
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/930
ping
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/930
No. I just wonder if this is not merged even if it passes CI for a long
time ago.
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1113
I just refactor NoteInterpreter to merge into InterpreterFactory. Could you
please rebase it to resolve conflict?
---
If your project is set up for it, you can reply to this email and have your
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1114
It's very helpful to edit your title with [DOC] prefix.
---
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 doe
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1025
ping
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/987
I'll make a new PR after I finish some preceding tasks.
---
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 doe
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/987
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1114
The failure is irrelevant. @fvaleri Could you please link the previous work
to refer it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1115
[ZEPPELIN-1099] Build and run Spark without spark-dependencies
### What is this PR for?
When users use SPARK_HOME, they won't want to include spark-dependencies.
This PR makes it pos
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1114
@Leemoonsoo Please review this PR. You were the reviewer of previous PRs.
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1115
Ready to 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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1112
Ready to 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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1121
It looks like a big change. Do you explain this idea in details? Is
`workflow` related job management?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1025
Is this PR completed?
---
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
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1016
Go for it!!
---
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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1025
It remains for the review, doesn't it?
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1122
Ready to review. 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 well. If your project doe
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1077
@rerngvit How about stopping Zeppelin, removing conf/interpreter.json,
starting Zeppelin and testing it again?
---
If your project is set up for it, you can reply to this email and have your
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1122
Rebased it
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1115
Rebased it
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1122
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 well. If your project does not have this fe
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1128
[HOTFIX] NPE when there's no properties field in interpreter-setting.json
### What is this PR for?
Fixing NPE while initializing interpreters from interpreter-setting
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1115
@felixcheung I think it has great benefit to include spark-dependencies by
default especially for newbie. Thus I think it's enough to make building
Zeppelin without spark-dependencies.
-
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
GitHub user jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1115
Merging 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 well. If your project does not have this fe
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1122
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1122
[MINOR] Removed InterpreterInfoSerializer and related codes
### What is this PR for?
Simplifying InterpreterFactory and reduce classes
### What type of PR is it
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1125
@cloverhearts Thanks for catching up it. 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
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1145
[ZEPPELIN-1012] Support Interpreter Aliases
### What is this PR for?
Supporting aliases for same interpreter group and enabling running two
interpreters with same groups.
### What
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1146
Test case would help to understand this change.
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1142
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo Thanks for the review, and because this is the first working
draft and I've changed a lot, some incompatibility and bug may exists. I'll fix
them.
Concerning name
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1154
I think we need to add test case for this. How about you?
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo Merged name and aliases, and enabled working with 0.6.0's
conf/interpreter.json. Concerning matching interpreters, I've test it with
0.6.0 binary. But there're some
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1157
@prabhjyotsingh Changes look good, but do you guide me to this PR with my
local machine? Does it have easy way to test it?
---
If your project is set up for it, you can reply to this email and
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1155
@prabhjyotsingh How to deal with it in case of getting error while
downloading deps? Just edit and save it again?
---
If your project is set up for it, you can reply to this email and have your
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
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 well. If your project does not have this fe
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1166
Added another location of metastore_db in .gitignore
### What is this PR for?
Avoiding tracking from metastore_db when you run ZeppelinServer in you IDE
### What type of PR is it
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1166
It's very simple PR and I'm merging it
---
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 th
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1163
@rconline What is `kinit`? I've tried to find a `kinit` in my CentOS, but I
couldn't find it. Is it fine?
---
If your project is set up for it, you can reply to this email and have
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1157
Doesn't it become potential problem to set hadoop version as 2.7.2? Is
there a way to fit the version of hadoop globally?
---
If your project is set up for it, you can reply to this emai
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1157
And It's not necessary to have hadoop dependencies in case not to depend on
hadoop environment like mysql and postresql. I think we need different way to
deal with it. I think it's
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1168
[MINOR] Add formatter
### What is this PR for?
Unifying the format of codes
### What type of PR is it?
[Feature]
### Todos
* [x] - Add formatter
* [x] - Convert
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1168
Ready to review. Could you please check Eclipse's one? Open
ZeppelinServer.java and reformat it and check if something's different or not.
---
If your project is set up for it, you ca
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1163
@rconline In case of not having `kinit`, there isn't a way to use kerberos,
right?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1144
LGTM. 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 well. If your project does not have
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/908
What's the status of this PR?
---
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
en
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1144
@voyageth Could you please change the title including [ZEPPELIN-1128]?
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo I've check the default binding issues and I will fix it.
---
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 jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1168
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1168
[MINOR] Add formatter
### What is this PR for?
Unifying the format of codes
### What type of PR is it?
[Feature]
### Todos
* [x] - Add formatter
* [x
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo Concerning `sql interpreter not found`, I'll fix that issue
with another one. I'll remove `jdbc` magic after I adopt this feature.
---
If your project is set up for i
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/908
@cloverhearts I've got it. Thanks.
---
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 jongyoul commented on the pull request:
https://github.com/apache/zeppelin/commit/8790ba97caf0a08f7d73a3dec6b583f572f18189#commitcomment-18232098
@Leemoonsoo I will test it tomorrow and fix it if it's necessary.
---
If your project is set up for it, you can rep
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1157
@prabhjyotsingh This PR is not passed. Why do you merge it? I think we need
to pass the CI even though we have some flaky tests. Could you please check it
again?
---
If your project is set up
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1155
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1168
@bzz Thanks for the suggestion. I think we need to update docs and it's
time to install Eclipse by myself in order to check the difference between
Eclipse and Intellij. Thanks Alex.
-
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1168
@bzz I've found Intellij doesn't export all of some configurations. We need
to enforce the format by configuring checkstyle.xml.
---
If your project is set up for it, you can rep
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1155
I also think @karup1990's way is proper.
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo I've reverted `refName` to `group` again. Review this PR
finally.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user jongyoul commented on the pull request:
https://github.com/apache/zeppelin/commit/8790ba97caf0a08f7d73a3dec6b583f572f18189#commitcomment-18247303
I've updated comments into ZEPPELIN-1169
---
If your project is set up for it, you can reply to this email and have
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1187
@zmhassan Hi zmhassan. As I mentioned in a JIRA issue, this is in progress
by @zjffdu. Could you concede? @zjffdu already contacted me for a few days ago
for this issue and I asked for him to
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1187
@zmhassan Thanks for understanding. ZEPPELIN-1012 makes a conflict and this
is for beginner thus I don't want for him/her to resolve conflict. @zjffdu
Thanks for understanding it.
---
If
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo I agree that this PR lower user experience in your case. It's
better to support that feature for backward compatibility. BTW, I also think
this makes users confused because use
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1145
@Leemoonsoo Thanks for detail 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 jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1199
ZEPPELIN-1169 Fix wrong Guava version
### What is this PR for?
Fixing the incompatible version for guava
### What type of PR is it?
[Hot Fix]
### Todos
* [x
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1199
@Leemoonsoo I've reverted guava version for original one.
---
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 doe
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1199
At first, revert the version and will find how to do selenium test via
Safari.
---
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 jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1200
[ZEPPELIN-1191] Supported legacy way to run paragraph with group name only
### What is this PR for?
Preserving legacy way to run paragraph when users use group name only
### What
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1199
---
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 jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1199
[HOTFIX][ZEPPELIN-1169] Fix wrong Guava version
### What is this PR for?
Fixing the incompatible version for guava
### What type of PR is it?
[Hot Fix]
### Todos
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1197
@SachinJanani Hi, can you add a test case for it? It would help understand
and test this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
201 - 300 of 1657 matches
Mail list logo