[GitHub] incubator-zeppelin pull request: [ZEPPELIN-828] Use individual use...

2016-05-21 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-220805129 @Leemoonsoo I agree. I removed HiveInterpreter changes from this PR. What do you think about using Entity instead of Data source? --- If your project

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-16 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/856 [ZEPPELIN-826] Run scheduled notes with a user ### What is this PR for? As a user, I would like to run scheduled note paragraphs with my user. This user information is passed

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-16 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/856 --- 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] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-12 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/856 [ZEPPELIN-826] Run scheduled notes with a user ### What is this PR for? As a user, I would like to run scheduled note paragraphs with my user. This user information is passed

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-12 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/856#issuecomment-218819264 I made the change to save user credentials in #860. Two instances of the build are failing due to the reasons below: 10548 Unable to extract spark

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-12 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/856 --- 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] incubator-zeppelin pull request: [ZEPPELIN-828] Use individual use...

2016-05-10 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-218261399 @Leemoonsoo I agree with you and have implemented the change to pass all user credentials to the interpreter. I also updated the screencast. Please let

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-841] Allow note creator...

2016-05-09 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/878 [ZEPPELIN-841] Allow note creators to define row level permissions ### What is this PR for? Allow note creators to define row level permissions. Here's an article

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-05-06 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/856#issuecomment-217516417 @Leemoonsoo I added "Save user credentials in a file with encryption" TODO in [#860](https://github.com/apache/incubator-zeppelin/pull/860).

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-819] Add 'Last updated ...

2016-05-02 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/848#issuecomment-216356985 @Leemoonsoo I made the change below from your PR 870 and the site works but I am still getting warnings in the log file. diff --git a/zeppelin

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-819] Add 'Last updated ...

2016-04-30 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/848#issuecomment-215987782 @Leemoonsoo I merged with latest master, fixed the jshint errors and tested with anonymous authentication. When I tried with basic authentication, I got

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-828] Use individual use...

2016-04-27 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/860 [ZEPPELIN-828] Use individual user credentials for data source authentication ### What is this PR for? The purpose of this pull request is to allow users to use their own

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-826] Run scheduled note...

2016-04-25 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/856 [ZEPPELIN-826] Run scheduled notes with a user ### What is this PR for? As a user, I would like to run scheduled note paragraphs with my user. This user information is passed to

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-798] Migrate to Jetty v...

2016-04-24 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-214041770 @echarles I tested this branch on my local machine but haven't deployed it to production yet. Since I deployed the workarounds I mentioned in

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-819] Add 'Last updated ...

2016-04-24 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/848 [ZEPPELIN-819] Add 'Last updated by X at Y' message in paragraph footer ### What is this PR for? In a large organization, it is helpful to know who updated a paragr

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-819] Add 'Last updated ...

2016-04-24 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/848 --- 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] incubator-zeppelin pull request: [ZEPPELIN-820] Reduce websocket c...

2016-04-22 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/850#issuecomment-213580839 @bzz New build succeeded. Created [ZEPPELIN-822](https://issues.apache.org/jira/browse/ZEPPELIN-822) - Flaky test

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-820] Reduce websocket c...

2016-04-22 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/850 [ZEPPELIN-820] Reduce websocket communication by unicasting instead of broadcasting note list ### What is this PR for? Reducing websocket communication by unicasting instead

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-820] Reduce websocket c...

2016-04-22 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/850 --- 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] incubator-zeppelin pull request: [ZEPPELIN-820] Reduce websocket c...

2016-04-21 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/850 [ZEPPELIN-820] Reduce websocket communication by unicasting instead of broadcasting note list ### What is this PR for? Reducing websocket communication by unicasting instead of

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-819] Add 'Last updated ...

2016-04-21 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/848 [ZEPPELIN-819] Add 'Last updated by X at Y' message in paragraph footer ### What is this PR for? In a large organization, it is helpful to know who updated a paragr

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-798] Migrate to Jetty v...

2016-04-12 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-209134954 Good work @echarles! I could run the zeppelin server my laptop with jetty 9.2 and jdk 1.8. We are using jdk 1.8 in production since we need bug fixes in

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-666] Change location of...

2016-03-13 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/766#issuecomment-196050734 Thanks @Leemoonsoo for the 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-666] Change location of...

2016-03-07 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/766 [ZEPPELIN-666] Change location of notebook authorization information. ### What is this PR for? Currently, notebook authorization information is stored in the note file

[GitHub] incubator-zeppelin pull request: Notebook Authorization

2016-02-26 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/681 --- 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] incubator-zeppelin pull request: Notebook Authorization

2016-02-26 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/681 Notebook Authorization ### What is this PR for? The goal of the PR is to add authorization for notebooks according to the design document [here] (https://gist.github.com

[GitHub] incubator-zeppelin pull request: Notebook Authorization

2016-02-17 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-185476274 Got it. Your suggestion to save notebook authorization information in a .ini file makes sense. In our company, we have an LDAP server with users

[GitHub] incubator-zeppelin pull request: Notebook Authorization

2016-02-15 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-184474829 @AhyoungRyu - I have fixed the documentation issues you reported. Thanks! @hayssams - Your design allows for fine-grained controls and is elegant

[GitHub] incubator-zeppelin pull request: Notebook Authorization

2016-02-10 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-182639353 @Leemoonsoo and @sourav-mazumder, I really appreciate your feedback. @Leemoonsoo, I fixed the note permission background and wildcard

[GitHub] incubator-zeppelin pull request: Notebook Authorization

2016-02-01 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/681 Notebook Authorization ### What is this PR for? The goal of the PR is to add authorization for notebooks according to the design document [here] (https://gist.github.com

[GitHub] incubator-zeppelin pull request: Add a (local mode) Scalding Inter...

2015-12-22 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/561#issuecomment-166755452 Should we add ScaldingInterpreter to ZEPPELIN_INTERPRETERS in ZeppelinConfiguration.java? tw-172-25-131-152 incubator-zeppelin (presto) $ git

[GitHub] incubator-zeppelin pull request: ZEPPELIN-487 Change supporting mu...

2015-12-08 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/517#issuecomment-162991822 Hi Jongyoul, since createStatement is implemented in the JDBC drivers, I think it is unlikely it will return a closed statement. However, I am all for

[GitHub] incubator-zeppelin pull request: ZEPPELIN-487 Change supporting mu...

2015-12-06 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/517#issuecomment-162349965 I tested the pull request and it allowed Vertica queries to execute in parallel with one modification. Thanks! I had to comment: //if