[GitHub] zeppelin issue #1187: [ZEPPELIN-1163] Change some parameter name properly in...

2016-09-01 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1187 Hi @zmhassan do you think you will have some time to rebase this PR and fix the indent? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1319: ZEPPELIN-1319 Use absolute path for ssl truststore and...

2016-09-01 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1319 @prabhjyotsingh Should we merge this? --- 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 pull request #1370: [ZEPPELIN-1379] Flink interpreter is missing sc...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1370 --- 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] zeppelin pull request #1377: ZEPPELIN-1384. Spark interpreter binary compati...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1377 --- 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] zeppelin pull request #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1358 --- 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] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-01 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1358 @astroshim You ping, I merge :) --- 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] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 Thank you @corneadoug --- 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] zeppelin issue #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

2016-09-01 Thread H4ml3t
Github user H4ml3t commented on the issue: https://github.com/apache/zeppelin/pull/1347 Hi @zjffdu, no I'm still using the release source for ``Affects Version/s: 0.6.1`` compiling it with Scala 2.10. You wrote in the issue description "``happens when SPARK_HOME is not defined``" but

[GitHub] zeppelin pull request #1392: WIP [ZEPPELIN-1339] Multi-user note management

2016-09-01 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1392 WIP [ZEPPELIN-1339] Multi-user note management ### What is this PR for? This PR addresses multi-user note management in Zeppelin. The idea is to apply filtering for notes per user base

[GitHub] zeppelin issue #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1347 @H4ml3t maybe I miss your case, but I think this PR should also resolve your case. Do you mind to try this PR ? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1390 this approach is a bit more breaking in terms of changes as well as compatibility with older versions (at least on notebook and storage level). i was working more on a filtering approach on

[GitHub] zeppelin pull request #1382: [ZEPPELIN-1391][Interpreters] print error while...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1382 --- 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] zeppelin issue #1319: ZEPPELIN-1319 Use absolute path for ssl truststore and...

2016-09-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1319 @corneadoug Yes, sure. Will merge this soon 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 pr

[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

2016-09-01 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 Thanks everyone! - Kavin --- 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

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

2016-09-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1387 @lresende I fixed the another wrong path issue and pushed the patch to your Zeppelin repo. Could you check [https://github.com/lresende/incubator-zeppelin/pull/4](https://github.com/lresende/in

[GitHub] zeppelin pull request #1369: [ZEPPELIN-1376] Add proxy credentials for depen...

2016-09-01 Thread doanduyhai
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/1369 [ZEPPELIN-1376] Add proxy credentials for dependency repo for corporate firewall use-cases ### What is this PR for? When using Zeppelin behind corporate firewall, sometimes the dependenc

[GitHub] zeppelin pull request #1369: [ZEPPELIN-1376] Add proxy credentials for depen...

2016-09-01 Thread doanduyhai
Github user doanduyhai closed the pull request at: https://github.com/apache/zeppelin/pull/1369 --- 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] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1302 @astroshim Tested again and it works well. I like it :) Here is the new UI (for the other reviewers) ![new](https://cloud.githubusercontent.com/assets/10060731/18161826/9a020222-706e-11

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Good. I am going to check first commit(proxy) soon. --- 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] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda Yeah it's good idea to apply github markdown style to Zeppelin. I tested again and it works well. Definitely better than before! But I found little bit wired behavior when I ma

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-01 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1393 [ZEPPELIN-1142] Zeppelin allows two users to simultaneously edit the notebook permissions ### What is this PR for? This is about storing the owner information on creating a note so when

[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-01 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1334 @bzz The following updates are made: 1 - Updating beam version from 0.1 to 0.2 2 - License report is updated for all dependencies Regarding point #2 in the previous message, do y

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

2016-09-01 Thread beriaanirudh
Github user beriaanirudh commented on the issue: https://github.com/apache/zeppelin/pull/1283 @corneadoug @Leemoonsoo please let me know if any action-item on my side. --- 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] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles also would you mind rebasing from master? --- 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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 @AhyoungRyu It was due to the `ANCHORLINK` option in pegdown. I removed it. ![markdown gif](https://cloud.githubusercontent.com/assets/4968473/18173464/a18a4d66-70a4-11e6-95b7-7d

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1390 @khalidhuseynov Good to discuss with you about what a multiuser note management would be. For the file system repo, this PR will give you: ``` ├── anonynmous │ Â

[GitHub] zeppelin issue #1379: [ZEPPELIN-1116]send out more exception msg

2016-09-01 Thread passionke
Github user passionke commented on the issue: https://github.com/apache/zeppelin/pull/1379 thanks your advice, It seems that the memory-based ByteArrayOutputStream do not need a close() method to release resources. Here is an answer at http://stackoverflow.com/questions/2330

[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1384 Regarding @felixcheung's comment, I think keeping backward compatibility is always very important. If i add more possible options after @1ambda 's - Create new interprete

[GitHub] zeppelin issue #1380: ZEPPELIN-1326: make profile to select dependency of ha...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1380 @jongyoul Configure a jdbc interpreter (e.g. with `mysql:mysql-connector-java:5.1.38` on it's dependency) and try to use it, following the official documentation. Then you can see the error.

[GitHub] zeppelin issue #1380: ZEPPELIN-1326: make profile to select dependency of ha...

2016-09-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1380 @Leemoonsoo I'll look into 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

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles Could you elaborate little bit about what will happen on the file system when notebook has multiple owners? Concept of per user interpreter-setting.json seems conflict wi

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1283 Tested again and got **Case2** works fine on this branch. LGTM. CI failure is irrelevant to this change. Merge if there're no more discussions. Thanks @beriaanirudh for the con

[GitHub] zeppelin pull request #1374: [MINOR] Add new line before logging paragraph c...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1374 --- 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] zeppelin issue #1381: [ZEPPELIN-1390] SparkInterpreter does not work for Spa...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1381 @tae-jun, could you trigger ci (close / reopen pullrequest will trigger ci) and see if it passes? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1285: Cache zeppelin-web/node_modules on travis ci and see i...

2016-09-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1285 I still see a lot of CI run fails on "npm install", so i'm merging this PR into master as a hotfix and see if it helps. --- If your project is set up for it, you can reply to this email and ha

[GitHub] zeppelin pull request #1285: Cache zeppelin-web/node_modules on travis ci an...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1285 --- 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

How to start Zeppelin By IDE

2016-09-01 Thread Hefei Li
Hi,all: I want to try start Zeppelin server in development mode by IDE (Eclipse or IDEA.). Now, can you give me some help and advice! -- = Hefei Li MP: +86 18701581473 MSN: lh...@live.cn ==

Re: How to start Zeppelin By IDE

2016-09-01 Thread moon soo Lee
Hi, Eclipse, i used to create eclipse project file by running "mvn eclipse:eclipse". IDEA, just importing maven project and it usually works without problem. You can also check http://zeppelin.apache.org/docs/latest/development/howtocontribute.html#integrated-development-environment-ide http://ze

[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-09-01 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1371 I have changed the logic of code a bit. Since it is just impossible to maintain a list of date formats for parsing purpose, now I just take whatever in the cell (if the value was proven not

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles Did you also implement interpreter setting per user in this PR as described in ZEPPELIN-1338 ? --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] zeppelin pull request #1381: [ZEPPELIN-1390] SparkInterpreter does not work ...

2016-09-01 Thread tae-jun
GitHub user tae-jun reopened a pull request: https://github.com/apache/zeppelin/pull/1381 [ZEPPELIN-1390] SparkInterpreter does not work for Spark2 version of HDP 2.5 ### What is this PR for? Spark2 version of HDP 2.5 is "2.0.0.2.5.0.0-1245". Currently, Zeppelin parses this vers

[GitHub] zeppelin pull request #1381: [ZEPPELIN-1390] SparkInterpreter does not work ...

2016-09-01 Thread tae-jun
Github user tae-jun closed the pull request at: https://github.com/apache/zeppelin/pull/1381 --- 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 #1381: [ZEPPELIN-1390] SparkInterpreter does not work for Spa...

2016-09-01 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1381 @Leemoonsoo Yep, I did 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

[GitHub] zeppelin issue #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

2016-09-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1360 I've rebased it because of resolving conflict. I'll merge it soon. --- 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] zeppelin pull request #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1360 --- 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] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai BTW how come only spark interpreter shows `error message bar`? --- 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 #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 @jongyoul I don't see the `per user` in the interpreter page. Do I miss anything ? ![image](https://cloud.githubusercontent.com/assets/164491/18191192/03daf4ba-70fb-11e6-9753-03a81a4bbd75.png)

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1265 It looks looks like a browser cache issue. Can you clean cache and try it again? On Friday, 2 September 2016, Jeff Zhang wrote: > @jongyoul I do

[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-01 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda Yeah tested again and it looks good👍 Thanks! (One more thing: [Zeppelin's Java block indentation size](https://google.github.io/styleguide/javaguide.html#s4.2-block-indentation

[GitHub] zeppelin pull request #1319: ZEPPELIN-1319 Use absolute path for ssl trustst...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1319 --- 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] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 Thanks @jongyoul after clean cache, I can see the `per user`. But unfortunately, two users still share the same interpreter. Here's what I see. I use yarn-client mode and only see one yarn app is

[GitHub] zeppelin issue #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1389 @AhyoungRyu Thank you!! :+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 a

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 Besides I have other 2 concerns. * I am a little confused about the combination of the interpreter options. There's so many combinations (2*2*3=12), I am not sure whether users are comfortable

[GitHub] zeppelin pull request #1394: rename r directory to 2BWJFTXKJ

2016-09-01 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1394 rename r directory to 2BWJFTXKJ ### What is this PR for? Rename "R" directory to "2BWJFTXKJ" under notebook directory to make it look like all other notebooks. ### What type o

[GitHub] zeppelin issue #1394: rename r directory to 2BWJFTXKJ

2016-09-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1394 @echarles I'm assuming there was no specific reason to keep name of this directory as "R" and hence, renaming this to look like any other notebook. --- If your project is set up for it, yo

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] Note and service...

2016-09-01 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1390 so build failed at least because of checkstyle violations. here are some to fix: ``` [ERROR] ZeppelinConfiguration.java[327] (sizes) LineLength: Line is longer than 100 characters (

Re: How to start Zeppelin By IDE

2016-09-01 Thread Kavin Kumar K S
We can use the below commands to run the zeppelin server in IDE. mvn antrun:run@start-zeppelin mvn antrun:run@stop-zeppelin Thanks, Kavin On Fri, Sep 2, 2016 at 2:32 AM, moon soo Lee wrote: > Hi, > > Eclipse, i used to create eclipse project file by running "mvn > eclipse:eclipse". > IDEA, jus

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1265 About the first comment you should set `scoped` or `isolated`, not `shared`. And growth of complexity is to remain backward compatibility. I also agree that interpreter setting become too m

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-01 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1302 My only feedback is that right now, it's hard to know if the interpreter has finished restarting or not. My guess is that it takes some time, so you shouldn't be able to restart it again an

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1265 @jongyoul Can you make a quick recap on the difference between all of those options? (perUser, perNote, shared, scopes, isolated) --- If your project is set up for it, you can reply to thi

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 Thanks @jongyoul , change it to `isolated` works. Regarding the option complexity, I still have concern on that. So now there's 2 dimensions for interpreter (user/note) IIUC. Before 0.6, zeppe

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 Proxy repo tested and it works properly. :+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