[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1231 @onkarshedge thanks for this contribution. as i can see, the design issue on passing `BittorrentNotebookRepo` to `TorrentSocket` was solved by passing through constructor. also better have

[GitHub] zeppelin issue #1251: [ZEPPELIN-1256][BUILD] Build distribution package with...

2016-08-02 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1251 @felixcheung I added description 99d91bf for the user who uses Spark version earlier than 2.0.0. And planning to add [zeppelin-spark_2.10 installation link](https://github.com/minahlee/zeppelin/

[GitHub] zeppelin issue #1255: Adding extra libraries to livy

2016-08-02 Thread jimdowling
Github user jimdowling commented on the issue: https://github.com/apache/zeppelin/pull/1255 Do you mean by modifying the UI on notebook creation or using a param to selecting the interpreter like '%livy --jars [path] --- If your project is set up for it, you can reply to this em

[GitHub] zeppelin issue #1255: Adding extra libraries to livy

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1255 No, I mean add interpreter property to allow user to specify jars/packages. --- 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 #673: [Zeppelin-630] Introduce new way of dependency loading ...

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/673 Thanks @minahlee, it works. But any reason to make it hidden by default ? But it is not intuitive for me that I can click the gear button to display it. --- If your project is set up for it, you c

[GitHub] zeppelin issue #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1210 @mwpenny Alright, so after taking a look, here are my feedbacks: Overall PR * License of angular-esri-map will need to be added in [this file](https://github.com/apac

[jira] [Created] (ZEPPELIN-1266) If the result of many paragraph, the code editor is very slow

2016-08-02 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1266: -- Summary: If the result of many paragraph, the code editor is very slow Key: ZEPPELIN-1266 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1266 Project: Zeppel

[GitHub] zeppelin pull request #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1258 [ ZEPPELIN-1266 ] Code editor Optimization ### What is this PR for? If the result of many para- graph, the code editor is very slow. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1258 ci trigger. --- 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, o

[GitHub] zeppelin pull request #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1258 [ ZEPPELIN-1266 ] Code editor Optimization ### What is this PR for? If the result of many para- graph, the code editor is very slow. ### What type of PR is it? Improvement

[GitHub] zeppelin pull request #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1258 --- 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 featur

[jira] [Created] (ZEPPELIN-1267) PySparkInterpreter doesn't work in spark 2.0

2016-08-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1267: Summary: PySparkInterpreter doesn't work in spark 2.0 Key: ZEPPELIN-1267 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1267 Project: Zeppelin Issue Typ

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1258 please, remove to browser cache --- 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] zeppelin pull request #1259: [ZEPPELIN-1260] R interpreter doesn't work with...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1259 [ZEPPELIN-1260] R interpreter doesn't work with Spark 2.0 ### What is this PR for? This PR fixes R interpreter doesn't work with Spark 2.0 ### What type of PR is it? Bug F

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1258 Would it make sense to change `isParagraphSaveAlready` => `wasParagraphAlreadySaved` ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1259 Looks great to me, modulo question above. Thank you for prompt fix @Leemoonsoo ! Shall this be merged to master or branch-0.6 as well? Asking as JIRA fix version is not set for this one.

[GitHub] zeppelin pull request #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work ...

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1260 ZEPPELIN-1267. PySparkInterpreter doesn't work in spark 2.0 ### What is this PR for? PySparkInterpreter doesn't work in spark 2.0 because pyspark and py4j is not distributed to executors. Thi

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1258 @bzz Thank you for your feedback. i will check to your opinion. Thank you --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 👍 for having a test! Looks good to me, modulo the issue raised above. --- 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

[GitHub] zeppelin issue #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work in spar...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1260 \cc @jongyoul 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 s

[jira] [Created] (ZEPPELIN-1268) As an enduser, I would like to embed paragraph and refresh the content of it in my custom UI

2016-08-02 Thread Kavin Kumar (JIRA)
Kavin Kumar created ZEPPELIN-1268: - Summary: As an enduser, I would like to embed paragraph and refresh the content of it in my custom UI Key: ZEPPELIN-1268 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1268

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1254 @khalidhuseynov thank you! Do you suggest merging this guy now and taking care of Revision separation later? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/1231 @khalidhuseynov thanks for helping. To actually see `TorrentSocket` in action I have to create a`WebSocketCreator` and a `Servlet`, then add this to jetty server , [at this line](htt

[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1231 @onkarshedge on the architecture side - could you please elaborate on the design that you choose, especially regarding `TorrentServlet` - why do you need a new servlet? What are the parts that communi

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-08-02 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1254 @bzz i believe we decided not to merge it for now. I'll revise it and let you know --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/1231 @bzz . I will post a diagram of communication for clear view. `BittorrentNotebookRepo` does not implement/override any `NotebookRepo` methods. It uses all VFSNotebookRepo methods. ```ja

[GitHub] zeppelin issue #1255: Adding extra libraries to livy

2016-08-02 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1255 I think that the livy itself doesn't support add (--jars/--packages property in spark-submit) --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] zeppelin issue #1255: Adding extra libraries to livy

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1255 You can use --conf spark.jars= and --conf spark.jars.packages= instead --- 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 proj

[GitHub] zeppelin issue #1227: [ZEPPELIN-1198][Spark Standalone] Documents for runnin...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1227 Looks great to me! Merging if there is no further discussion. > Does this needs documentation? no I think it does ;) --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1259 @bzz Thanks for quick review. Updated the indent. This should be merged to branch-0.6 as well --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1261 [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Failed to create interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA ### What is this PR for? Fix CI test failure wi

[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1248 tested. 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 issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1261 CI seems to not beeing triggered on this branch :\ Trying locally with `time mvn package -pl "\!zeppelin-distribution"` --- If your project is set up for it, you can reply to this email and h

[GitHub] zeppelin pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/1261 --- 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 pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/1261 [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Failed to create interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA ### What is this PR for? Fix CI test failure

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/932 @Leemoonsoo Just pushed the asked changes regarding licenses. --- 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] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1261 Not sure why CI is not triggered in this PR. Here's CI result with my travis account https://travis-ci.org/Leemoonsoo/zeppelin/builds/149153003 CI is not green yet, but at least ZEP

[GitHub] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1261 Local build results are - it hangs for last 4 minutes with the error below ``` 20:56:59,101 INFO org.apache.zeppelin.notebook.Paragraph:282 - run paragraph 20160802-205659_1961452200

[GitHub] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1261 Have updated the comment, you might need to reload the page --- 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 hav

[GitHub] zeppelin issue #1234: [ZEPPELIN-1234] Fix for bug ZEPPELIN-1234

2016-08-02 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/1234 Travis checks are failing but are not related to this changes.@jongyoul Can you please have a look. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1258 I would limit this PR to the first commit, for me the rest is out of scope. Also, in the first PR, there is no need to make a if before calling `setParagraphMode`, the conditions you added i

[GitHub] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1261 @bzz I see the same error on my local test. However somehow travis does not have the error. Anyway, do you mind file a new issue for the new error? --- If your project is set up for it, yo

[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1259 @minahlee Just saw you comment on ZEPPELIN-1260, but @Leemoonsoo has been fast... I have tested the changes, and now sparkr dataframes are working well (with the `spark` variable which r

[GitHub] zeppelin pull request #1262: [ZEPPELIN-1268]

2016-08-02 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1262 [ZEPPELIN-1268] ### What is this PR for? This pull request is to add the below options so when someone develops a custom UI it would ease them. 1. Add a new option "Get paragraph id"

[GitHub] zeppelin issue #1239: [HOTFIX][ZEPPELIN-1240] Removed interpreter properties...

2016-08-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1239 @Leemoonsoo @minahlee Could you please test 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

[GitHub] zeppelin issue #1262: [ZEPPELIN-1268]

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1262 @kavinkumarks Thanks for the contribution, the Paragraph ID is actually already shown in the UI https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragr

[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1259 Thanks @echarles for the review. I have exposed sqlContext as 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 projec

[GitHub] zeppelin pull request #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/zeppelin/pull/1263 Small cleanup of zeppelin-server tests ### What is this PR for? Small cleanup of zeppelin-server tests ### What type of PR is it? Refactoring ### How should this be tested?

[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1231 @onkarshedge >> Initial idea the we discussed was that `BittorrentNotebookRepo` encapsulates all the things that are required in order to get notebooks. > Could you please elaborate this

[jira] [Created] (ZEPPELIN-1269) HeliumApplicationFactoryTest hangs on default build from master

2016-08-02 Thread Alexander Bezzubov (JIRA)
at org.apache.thrift.transport.TSocket.open(TSocket.java:187) .. 20:56:59,101 INFO org.apache.zeppelin.notebook.Paragraph:282 - run paragraph 20160802-205659_1961452200 using null org.apache.zeppelin.interpreter.LazyOpenInterpreter@747b96ad 20:56:59,105 ERROR org.apache.zeppelin.schedule

[GitHub] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1261 Have created [ZEPPELIN-1269](https://issues.apache.org/jira/browse/ZEPPELIN-1269) LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin pull request #1264: Updated path to point to latest Maven binaries

2016-08-02 Thread jojurajan
GitHub user jojurajan opened a pull request: https://github.com/apache/zeppelin/pull/1264 Updated path to point to latest Maven binaries ### What is this PR for? To update link of Maven binary provided in the installation steps. ### What type of PR is it? [Documentat

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 ## CI Failures in This PR Profile [2](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308638/log.txt) Hangs a la [ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPE

[GitHub] zeppelin pull request #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo closed the pull request at: https://github.com/apache/zeppelin/pull/928 --- 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 pull request #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
GitHub user rawkintrevo reopened a pull request: https://github.com/apache/zeppelin/pull/928 [ZEPPELIN-116] Add Apache Mahout Interpreter ### What is this PR for? This PR adds Mahout functionality for the Spark Interpreter. ### What type of PR is it? Improvement

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

2016-08-02 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1265 [ZEPPELIN-1210] Run interpreter per user ### What is this PR for? Enabling each user to run same interpreter. ### What type of PR is it? [Improvement] ### What is the Ji

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 [Build 6281](https://travis-ci.org/apache/zeppelin/builds/149199571) Profiles [1](https://s3.amazonaws.com/archive.travis-ci.org/jobs/149199572/log.txt), [2](https://s3.amazonaws.com/a

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-08-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1247 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

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

2016-08-02 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1265 Quickly went through the changes, but not sure to get it. In which way is this related to the various intepreter modes we already have? In which way is this related to ZEPPELIN-1000 (Multi

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/932 Failing with org.apache.zeppelin.interpreter.InterpreterException: org.apache.thrift.TApplicationException: Internal error processing createInterpreter at org.apache.zeppelin.in

[GitHub] zeppelin issue #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1259 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 fe

[GitHub] zeppelin pull request #1266: [ZEPPELIN-1219] Add searching feature to Zeppel...

2016-08-02 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1266 [ZEPPELIN-1219] Add searching feature to Zeppelin docs site ### What is this PR for? As more and more document pages are added, it's really hard to find specific pages. So I added searchin

[GitHub] zeppelin issue #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Fail...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1261 Merge it into master and branch-0.6 as a 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 thi

[GitHub] zeppelin pull request #1261: [ZEPPELIN-1264] [HOTFIX] Fix CI test failure wi...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1261 --- 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 #1259: [ZEPPELIN-1260] R interpreter doesn't work with Spark ...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1259 Merge it into master and branch-0.6 if there're 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 projec

[GitHub] zeppelin pull request #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplot...

2016-08-02 Thread bustios
GitHub user bustios opened a pull request: https://github.com/apache/zeppelin/pull/1267 [ZEPPELIN-1261] Bug fix in z.show() for matplotlib graphs ### What is this PR for? Bug fix in z.show() for matplotlib graphs and code refactoring ### What type of PR is it? Bug Fi

[GitHub] zeppelin issue #1260: ZEPPELIN-1267. PySparkInterpreter doesn't work in spar...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1260 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 feature enabled and wishes s

[GitHub] zeppelin issue #1232: ZEPPELIN-1197. Should print output directly without in...

2016-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1232 Merge into master 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 does not have

[GitHub] zeppelin pull request #1268: minor doc fix for r.md

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1268 minor doc fix for r.md ### What is this PR for? straightforward doc fix ### What type of PR is it? [Documentation] ### Todos * [ ] - Task ### What is the

[GitHub] zeppelin issue #1268: minor doc fix for r.md

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1268 @AhyoungRyu Please help review. 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 your project does not have this feature en

[jira] [Created] (ZEPPELIN-1270) Remove getting SQLContext from SparkSession.wrapped()

2016-08-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1270: Summary: Remove getting SQLContext from SparkSession.wrapped() Key: ZEPPELIN-1270 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1270 Project: Zeppelin

[GitHub] zeppelin issue #1269: ZEPPELIN-1270. Remove getting SQLContext from SparkSes...

2016-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1269 \cc @Leemoonsoo Please help review, 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 your project does not have this featur

[GitHub] zeppelin pull request #1269: ZEPPELIN-1270. Remove getting SQLContext from S...

2016-08-02 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1269 ZEPPELIN-1270. Remove getting SQLContext from SparkSession.wrapped() ### What is this PR for? SparkSession.wrapped is only for spark 2.0 preview, it is not supported in spark 2.0 release. So I

[GitHub] zeppelin issue #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1267 Looks great to me, thank you @bustios ! Merging to master if there is no further discussion --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] zeppelin issue #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1263 CI failed due to [ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264) Rebasing on latest master and merging, if there is no further discusison --- If your project is set up f

[jira] [Created] (ZEPPELIN-1271) ZeppelinSparkClusterTest.sparkRTest test is flaky and fails often

2016-08-02 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1271: Summary: ZeppelinSparkClusterTest.sparkRTest test is flaky and fails often Key: ZEPPELIN-1271 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1271

[GitHub] zeppelin issue #1266: [ZEPPELIN-1219] Add searching feature to Zeppelin docs...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1266 Great improvement @AhyoungRyu ! Posted few suggestions above. --- 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 h

[jira] [Created] (ZEPPELIN-1272) Zeppelin spark innterpreters throws java.lang.StackOverflowError after running a while

2016-08-02 Thread lichenglin (JIRA)
lichenglin created ZEPPELIN-1272: Summary: Zeppelin spark innterpreters throws java.lang.StackOverflowError after running a while Key: ZEPPELIN-1272 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1272

[GitHub] zeppelin issue #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bustios
Github user bustios commented on the issue: https://github.com/apache/zeppelin/pull/1267 I see. Thanks for your help and information @bzz. I'll keep it in mind for subsequent PRs --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/932 A quick search over JIRA suggests that the failure you reffer was taked care of in [ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264) and is resolved in latest master, please rebase.

[GitHub] zeppelin issue #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplotlib gra...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1267 Sounds great! Apparently CI failure has been addressed in #1259 as 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

[GitHub] zeppelin issue #1268: minor doc fix for r.md

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1268 @zjffdu you have good eyes :) 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 feat

[GitHub] zeppelin pull request #1259: [ZEPPELIN-1260] R interpreter doesn't work with...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1259 --- 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 #1268: minor doc fix for r.md

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1268 --- 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 #1207: [DOC][ZEPPELIN-1209] Remove a useless sentence about d...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1207 CI build failure is irrelevant with this PR i think. Can this be merged 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

[jira] [Created] (ZEPPELIN-1273) d3.format not called for large negative numbers

2016-08-02 Thread Prasad Wagle (JIRA)
Prasad Wagle created ZEPPELIN-1273: -- Summary: d3.format not called for large negative numbers Key: ZEPPELIN-1273 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1273 Project: Zeppelin Is

[GitHub] zeppelin pull request #1270: [ZEPPELIN-1273] Use Math.abs to determine if cu...

2016-08-02 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/zeppelin/pull/1270 [ZEPPELIN-1273] Use Math.abs to determine if custom formatter should be called for y axis values ### What is this PR for? Use Math.abs to determine if custom formatter should be called fo

[GitHub] zeppelin issue #1266: [ZEPPELIN-1219] Add searching feature to Zeppelin docs...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1266 @bzz Really appreciate for your all precise comments! Updated several docs as you suggested :) --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 I refactor method name. --- 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 #1267: [ZEPPELIN-1261] Bug fix in z.show() for matplot...

2016-08-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1267 --- 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 #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1258 @corneadoug I did modification. Please review for this pr. --- 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] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1258 @bzz I removed the parts which you say. Thank you. --- 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 doe

[GitHub] zeppelin issue #1263: Small cleanup of zeppelin-server tests

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1263 CI failure ``` Results : Failed tests: ZeppelinSparkClusterTest.sparkRTest:116 expected:<[[1] 3]> but was:<[Error in getSparkSession(): SparkSession not initialized

[jira] [Created] (ZEPPELIN-1274) Write "Spark SQL" in docs rather than "SparkSQL"

2016-08-02 Thread Kousuke Saruta (JIRA)
Kousuke Saruta created ZEPPELIN-1274: Summary: Write "Spark SQL" in docs rather than "SparkSQL" Key: ZEPPELIN-1274 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1274 Project: Zeppelin

[GitHub] zeppelin pull request #1271: [Z

2016-08-02 Thread sarutak
GitHub user sarutak opened a pull request: https://github.com/apache/zeppelin/pull/1271 [Z ### What is this PR for? Some of the doc files say "SparkSQL" but the collect spelling is "Spark SQL" (need a white space between "Spark" and "SQL"). Lets's replace it with the collect

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1258 Looks great 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 feature enabled and wishes so, or

[GitHub] zeppelin issue #1270: [ZEPPELIN-1273] Use Math.abs to determine if custom fo...

2016-08-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1270 Looks great to me, thank you @praagarw Merging to only to master, if there is no further discussion Or do you think we should also apply this to upcoming 0.6.1 as well? --- If your proje

[GitHub] zeppelin issue #1258: [ ZEPPELIN-1266 ] Code editor Optimization

2016-08-02 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1258 @cloverhearts Thanks for the changes and keeping this PR focused. I saw some great improvements with big Notebook with those changes. @Leemoonsoo Can you try it too and tell us if you als

[GitHub] zeppelin issue #1271: [ZEPPELIN-1274]Write "Spark SQL" in docs rather than "...

2016-08-02 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1271 @sarutak Thanks for fixing this. I found more `SparkSQL` in the below 2 files. - [docs/manual/dynamicform.md#using-form-templates](https://github.com/apache/zeppelin/blob/master/docs/manu

[GitHub] zeppelin issue #1271: [ZEPPELIN-1274]Write "Spark SQL" in docs rather than "...

2016-08-02 Thread sarutak
Github user sarutak commented on the issue: https://github.com/apache/zeppelin/pull/1271 @AhyoungRyu Thanks for the notification. I'll fix them 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 does no

  1   2   >