Mina Lee created ZEPPELIN-1304:
--
Summary: Prevent creating new interpreter setting with empty
interpreter name
Key: ZEPPELIN-1304
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1304
Project: Zeppeli
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1299
[ZEPPELIN-1304] Show popup when interpreter name is empty
### What is this PR for?
Prevent creating interpreter with empty name.
### What type of PR is it?
Bug Fix
### W
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1255
could you update the title of this PR and JIRA ZEPPELIN-1258, if we are
changing how we would approach this?
---
If your project is set up for it, you can reply to this email and have your
re
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1269
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1294
This seems to overlap with PR #1264?
---
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 mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1255
@felixcheung I suggests this title "Adding packages to livy interpreter",
Is it Ok ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1291
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 yoonjs2 commented on the issue:
https://github.com/apache/zeppelin/pull/1294
@felixcheung Thanks for comment but this PR (#1294) for vagrant script and
its documentation under scripts/vagrant/zeppelin-dev. PR #1264 for root
README.md about How to build Zeppelin. It may con
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1284
great improvement - would it be better if the tag matches the interpreter
name - it could be `%spark1` `%spark2` for example?
---
If your project is set up for it, you can reply to this email
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1299
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1294
I think it'd be great if you update the PR title and PR description. 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
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1255
how about "Add Spark packages support to Livy interpreter"?
---
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
Github user mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1255
@felixcheung 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 s
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA_OPTS
##
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1269
@Leemoonsoo PR rebased.
---
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 wishe
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1284
@felixcheung Thank you for your review.
I think The meta name of auto completion should be original meta name
because the user interpreters name that you mentioned(spark1,spark2) is not
rel
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@Leemoonsoo Yes, I agree with you.
I will change that. Thanks for 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
Jeff Zhang created ZEPPELIN-1305:
Summary: Fix bug of ZEPPELIN-1215
Key: ZEPPELIN-1305
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1305
Project: Zeppelin
Issue Type: Bug
R
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1300
ZEPPELIN-1305. Fix bug of ZEPPELIN-1215
### What is this PR for?
It is my bad, Before commit ZEPPELIN-1215, I change
`getProperty().getProperty(key) != null` to `getProperty().contains(key)`.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1300
\cc @prabhjyotsingh
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1269
LGTM and merge 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 this feat
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1280
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1279
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1269
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1291
---
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/1278
Looks like test is not passing
```
---
T E S T S
---
Running org.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1271
---
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/1270
merge into master and branch-0.6 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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1264
Thanks @jojurajan for the contribution!
Looks good to me.
---
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 projec
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1189
Change looks good.
Could you trigger CI once again and see if it becomes green?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Shim HyungSung created ZEPPELIN-1306:
Summary: Interpreter restarts on a note.
Key: ZEPPELIN-1306
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1306
Project: Zeppelin
Issue Type: Im
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA_OPTS
##
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1293
`LOG.debug` sounds like a great idea to me to avoid filling up the logs in
default configuration.
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA_OPTS
##
Anthony Corbacho created ZEPPELIN-1307:
--
Summary: Implement notebook revision in Zeppelinhub repo
Key: ZEPPELIN-1307
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1307
Project: Zeppelin
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1278
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1278
ZEPPELIN-1287. No need to call print to display output in PythonInterpreter
### What is this PR for?
It is not necessary to call print to display output in PythonInterpreter. 2
main changes:
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1300
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1300
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 feature
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1270
Let's merge! :Shipit:
---
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 WeichenXu123 commented on the issue:
https://github.com/apache/zeppelin/pull/1296
cc @zjffdu Thanks! It is an obvious bug.
---
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1300
LGTM
Merging this if 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 th
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1300
Considering progress of
[ZEPPELIN-1177](https://issues.apache.org/jira/browse/ZEPPELIN-1177), It's
about to start a release process of 0.6.1.
Let's merge this PR into master and branch-0.6
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1270
---
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/1230
Following the discussion in that JIRA issue, I'm not convinced this button
is necessary.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1288
---
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/1288
The second: https://travis-ci.org/apache/zeppelin/builds/149987385
There're flaky tests in this trial
```
Failed tests:
ZeppelinSparkClusterTest.pySparkTest:141 expecte
GitHub user jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/1288
[WIP]Fix the CI
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide -
https://github.com/a
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1286
Thanks @khalidhuseynov for the 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 y
Github user jojurajan commented on the issue:
https://github.com/apache/zeppelin/pull/1264
@Leemoonsoo You are welcome.
---
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 an
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1247
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1248
Do you check the error of CI?
```
Failed tests:
NoteTest.insertParagraphWithLastReplName:159 expected:<%spark > but
was:
NoteTest.addParagraphWithLastReplName:144 expecte
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Could you please revise the title first?
---
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1257
Except for minor fixes, looks good.
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1284
hmm, shouldn't we match the group name? that's what the user type though
(eg. `%spark1`) and auto complete is to help with the user when she's typing?
---
If your project is set up for it, y
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/1301
[MINOR] Removed unused profiles from spark/pom.xml
### What is this PR for?
Making spark/pom.xml simple
### What type of PR is it?
[Refactoring]
### Todos
* [x] - Si
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/1288
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1293
@khalidhuseynov Could you resolve conflicts?
---
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 kavinkumarks commented on the issue:
https://github.com/apache/zeppelin/pull/1262
Have updated the title, please let me know if I am missing anything else.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
62 matches
Mail list logo
