[GitHub] zeppelin issue #986: [Zeppelin 946] Permissions not honoring group

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

[GitHub] zeppelin issue #996: [ZEPPELIN-890] Cache downloaded file from 'maven-downlo...

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

[GitHub] zeppelin pull request #998: Update api/pass auth info to repo

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

[GitHub] zeppelin issue #997: [MINOR] Make nav zeppelin version to point ZEPPELIN_VER...

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

[GitHub] zeppelin issue #965: [ZEPPELIN-957] Normalize note name.

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

[GitHub] zeppelin issue #965: [ZEPPELIN-957] Normalize 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

[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

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] zeppelin pull request #996: [ZEPPELIN-890]

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

[GitHub] zeppelin pull request #995: [ZEPPELIN-990] Add header anchor for Zeppelin do...

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] zeppelin issue #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completion

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

[GitHub] zeppelin issue #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completion

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

[GitHub] zeppelin issue #994: [WIP][ZEPPELIN-986] Create publish release script

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

[GitHub] zeppelin issue #965: [ZEPPELIN-957] Normalize note name.

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] zeppelin issue #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completion

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] zeppelin issue #966: [ZEPPELIN-958] Support syntax highlight for python and ...

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:

[GitHub] zeppelin issue #980: [ZEPPELIN-871] [WIP] spark 2.0 interpreter on scala 2.1...

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

[jira] [Created] (ZEPPELIN-989) Zeppelin Cassandra can not connect

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] zeppelin pull request #966: [ZEPPELIN-958] Support syntax highlight for pyth...

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

[jira] [Created] (ZEPPELIN-988) java.lang.NoSuchMethodError: com.datastax.driver.core.ProtocolOptions.getProtocold

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:

[GitHub] zeppelin issue #992: [ZEPPELIN-980] missing "incubation-" references

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] zeppelin pull request #992: [ZEPPELIN-980] missing "incubation-" references

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] zeppelin issue #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completion

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

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

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

[GitHub] zeppelin pull request #983: [ZEPPELIN-980] Move git repository from incubato...

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

[GitHub] zeppelin issue #964: filtering subdirectory names.

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

[GitHub] zeppelin issue #983: [ZEPPELIN-980] Move git repository from incubator-zeppe...

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

[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

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] zeppelin pull request #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completi...

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

[GitHub] zeppelin pull request #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completi...

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 ###

[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

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] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

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

[GitHub] zeppelin pull request #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completi...

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

[GitHub] zeppelin pull request #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completi...

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 ###

[GitHub] zeppelin issue #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completion

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

[GitHub] zeppelin pull request #988: ZEPPELIN-985 ] Fixed bug in the Pyspark completi...

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 ###

[GitHub] zeppelin issue #984: [ZEPPELIN-982] Improve interpreter completion API

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

[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

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;>

[GitHub] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

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

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

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

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

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

[GitHub] zeppelin issue #964: filtering subdirectory names.

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

[GitHub] zeppelin pull request #987: ZEPPELIN-694 Remove ClassLoaderInterpreter

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] ###

[GitHub] zeppelin pull request #984: [ZEPPELIN-982] Improve interpreter completion AP...

GitHub user AhyoungRyu reopened a pull request: https://github.com/apache/zeppelin/pull/984 [ZEPPELIN-982] Improve interpreter completion API ### What is this PR for? When people implement a new interpreter, they extend

[GitHub] zeppelin issue #983: [ZEPPELIN-980] Move git repository from incubator-zeppe...

Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/983 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

[GitHub] zeppelin pull request #986: [Zeppelin 946] [WIP] Permissions not honoring gr...

GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/986 [Zeppelin 946] [WIP] Permissions not honoring group ### What is this PR for? Error: Insufficient privileges to write notebook. Allowed users or roles: [admin, zeppelinWrite]

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/979 tested. LGTM +1 --- 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