Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
Ready 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 project does not have this feature
enabled and wishes
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/996
One more suggestion not sure if that is relevant to this PR should we have
`5` near
https://github.com/apache/zeppelin/blob/master/spark-dependencies/pom.xml#L827
and
https://github.co
Github user chbrown commented on the issue:
https://github.com/apache/zeppelin/pull/747
I just tried out the `scala-210-211-rebased-moon` branch from
`lresende/incubator-zeppelin.git` and it seems to be working, using Spark 1.6.1
built from source for Scala 2.11.8, but I ran into seve
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/998
Update api/pass auth info to repo
### What is this PR for?
This PR is to make available information on the currently authenticated
user(subject) in the Notebook storage level. It can be
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/955
Thanks @SachinJanani for the contribution.
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 w
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/995
Cool! working well. 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 wi
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/950
---
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 en
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/997
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 f
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/997
[MINOR] Make nav zeppelin version to point ZEPPELIN_VERSION in _config.yml
### What is this PR for?
One less line to change when releasing docs by making `ZEPPELIN_VERSION` to
be retrieved fr
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/965
Thanks @astroshim for the improvement. I've tested and although it's
working well, i think there're little more things we can think
1. Normalization is not applied when change note name.
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/965
Is normalization applied on note name 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 f
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
Okay. I am going to fix it.
I show real interpreter name instead of '%..'
@jongyoul I think about single responsibility. But if you have thought it
is redundant, I will change code.
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/965
@felixcheung Thank you for pointing out that i missed. I just fixed 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 pro
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/996
[ZEPPELIN-890]
### What is this PR for?
Cache downloaded file from 'maven-download-plugin' to improve CI build
speed and reduce build failure due to network issue.
### What t
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/995
[ZEPPELIN-990] Add header anchor for Zeppelin documentation site
### What is this PR for?
Sometimes we want to share some docs title link as github markdown page
does. This PR enables this
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/988
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 featu
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/988
Tested and it works well.
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
enab
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/994
CI failiure is unrelated. 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 project does not have this feat
Ahyoung created ZEPPELIN-990:
Summary: Add header anchor for Zeppelin documentation site
Key: ZEPPELIN-990
URL: https://issues.apache.org/jira/browse/ZEPPELIN-990
Project: Zeppelin
Issue Type: Im
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/979
Tested, looks good. How about applying @corneadoug's opinion making the
first paragraph not to show `Move Up`, last paragraph not to show `Move Down`?
---
If your project is set up for it, you ca
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/965
should this check for both `/` and `\` then for windows?
https://github.com/apache/zeppelin/pull/965/files#diff-bcbdda87aa29314ed459e8212067e313R528
---
If your project is set up for it,
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/971
I would prefer to keep `noteAction` size as default height. As a Zeppelin
user I prefer to see more of my work space than notebook name and buttons.
---
If your project is set up for it, you can
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/988
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/966
right, I think that can be added to interpreterSettings and checked on the
frontend similar to this:
https://github.com/apache/zeppelin/blob/09bc8a36a437a2a37736449e330a7594c89f495d/zeppeli
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/980
From what I see, we should still be able to use most of the existing code.
Could you elaborate more on how bad it would be to support Spark 1.x and 2.x in
the same interpreter code?
---
If y
jesse dudley created ZEPPELIN-989:
-
Summary: Zeppelin Cassandra can not connect
Key: ZEPPELIN-989
URL: https://issues.apache.org/jira/browse/ZEPPELIN-989
Project: Zeppelin
Issue Type: Bug
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/966
---
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 en
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/994
[WIP][ZEPPELIN-986] Create publish release script
### What is this PR for?
This PR is to automate release publish to maven repository.
We used to use maven-deploy-plugin and maven-release-
Matt created ZEPPELIN-988:
-
Summary: java.lang.NoSuchMethodError:
com.datastax.driver.core.ProtocolOptions.getProtocold
Key: ZEPPELIN-988
URL: https://issues.apache.org/jira/browse/ZEPPELIN-988
Project: Zeppe
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/993
[ZEPPELIN-987] Enable user to secure interpreter setting, credentials and
configurations info
### What is this PR for?
For some user case, people might want to hide **Interpreter Setting**,
Ahyoung created ZEPPELIN-987:
Summary: Enable user to secure interpreter setting, credentials
and configurations info
Key: ZEPPELIN-987
URL: https://issues.apache.org/jira/browse/ZEPPELIN-987
Project: Zep
Mina Lee created ZEPPELIN-986:
-
Summary: Make custom maven publish release script
Key: ZEPPELIN-986
URL: https://issues.apache.org/jira/browse/ZEPPELIN-986
Project: Zeppelin
Issue Type: Improveme
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/984
@jongyoul Makes sense. I addressed your feedback. Please see my last two
commit :)
Regarding `value`,
as you can see, I used two variables(`name` and `value`) for
`InterpreterCompl
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/955
Can we merge this if there are no more comments?
---
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 t
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/992
Thanks for catching 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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/978
LGTM, I noticed you are clearing search box on click on any link ð
---
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 prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/992
[ZEPPELIN-980] missing "incubation-" references
### What is this PR for?
This is extension to https://github.com/apache/zeppelin/pull/983, with
missing references of incubation.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/zeppelin/pull/991
---
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 featu
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/991
Closing this, missed https://issues.apache.org/jira/browse/ZEPPELIN-980
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/988
@cloverhearts I tested and it works well ð
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/988
@cloverhearts Thank you for quick fixing ! I closed my issue
[ZEPPELIN-984](https://issues.apache.org/jira/browse/ZEPPELIN-984) since they
are duplicated.
---
If your project is set up for it,
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug @minahlee I fixed the code to hide "Remove" menu on the last
paragraph menu. please review.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/983
---
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 en
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/964
@corneadoug I refactored the codes that you pointed out. please 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 pro
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/983
Merging it as hotfix
---
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 prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/991
fix ci-image
### What is this PR for?
CI image/status for our github page is broken.
### What type of PR is it?
[Hot Fix]
### Todos
* [x] - replace `incuba
Github user onkarshedge commented on the issue:
https://github.com/apache/zeppelin/pull/989
@bzz please 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
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/990
[MINOR] Removed classloaderInterpreter on RemoteInterpreterServer
### What is this PR for?
Removed usage of ClassloaderInterpreter while RemoteInterpreter is being
created.
### What
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/988
---
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 cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/988
ZEPPELIN-985 ] Fixed bug in the Pyspark completion
### What is this PR for?
Currently, does not work 'pyspark completion'.
### What type of PR is it?
Bug Fix
### To
Github user onkarshedge commented on the issue:
https://github.com/apache/zeppelin/pull/989
@bzz please 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
Github user onkarshedge commented on the issue:
https://github.com/apache/zeppelin/pull/989
@bzz Please 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
GitHub user onkarshedge opened a pull request:
https://github.com/apache/zeppelin/pull/989
Added ipfsnotebookrepo [ Zeppelin-683 WIP ]
### What is this PR for?
IPFSNoteBookRepo IPFS storage and sharing zeppelin notes.
### What type of PR is it?
Feature
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/988
---
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 cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/988
ZEPPELIN-985 ] Fixed bug in the Pyspark completion
### What is this PR for?
Currently, does not work 'pyspark completion'.
### What type of PR is it?
Bug Fix
### To
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/988
retry ci
---
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 cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/988
ZEPPELIN-985 ] Fixed bug in the Pyspark completion
### What is this PR for?
Currently, does not work 'pyspark completion'.
### What type of PR is it?
Bug Fix
### Todo
CloverHearts created ZEPPELIN-985:
-
Summary: does not working pyspark keyword completion.
Key: ZEPPELIN-985
URL: https://issues.apache.org/jira/browse/ZEPPELIN-985
Project: Zeppelin
Issue Typ
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/984
@AhyoungRyu What the `value` of InterpreterCompletion means? And I suggest
we remove the mark abstract from Interpreter class because almost interpreter
returns empty list and it's better to offer
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/806
It works well with another default interpreter except `%spark`. See the
below.
https://cloud.githubusercontent.com/assets/3612566/15959490/0720f9e6-2f36-11e6-9f8b-ad6f627c4582.png";>
https:
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/806
@mwkang I still get `%..` when I create a new note, is it normal?
Could you also add a space after the %interpreter when you add it to the
note?
That way we can start typing directly
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
Thank you for clarification.
---
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 corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/979
I guess both methods can cohabitate then
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@minahlee @corneadoug Then we need two ways to solve this issue?
---
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 do
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/979
@minahlee Maybe then in that case it would be needed
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/964
@corneadoug Thank you for your review.
I'll refactor the code as you mentioned.
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/987
ZEPPELIN-694 Remove ClassLoaderInterpreter
### What is this PR for?
Removing some redundant codes for simplifying Zeppelin.
### What type of PR is it?
[Refactoring]
### T
67 matches
Mail list logo