[GitHub] incubator-zeppelin pull request:

2015-08-21 Thread djoelz
Github user djoelz commented on the pull request: https://github.com/apache/incubator-zeppelin/commit/d5ab911bf4419fa7c6f38945c6c8ad4946f8abf6#commitcomment-12841349 There are new fixes on top of this that might address the issue. Try and get the latest. Thanks! --- If your proj

[GitHub] incubator-zeppelin pull request:

2015-08-21 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/commit/acda3de9d8f36cd63e23d94f61484282e9ad0ef7#commitcomment-12841130 I'm now getting a white page for 5-10s with disconnected state before I get the list of notebooks and connected sta

[GitHub] incubator-zeppelin pull request: ZEPPELIN-157: Adding Map Visualiz...

2015-08-21 Thread Madhuka
Github user Madhuka commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-133631065 @eranwitkon It is been fixed few days ago. Please update your copy. Now there is data validator. So when you pass 1 column test will fail. Since Zeppelin

[GitHub] incubator-zeppelin pull request: ZEPPELIN-171,Add Apache Kylin int...

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/207#issuecomment-133628342 My apologies. Just before merge this branch, i realized it does not contain a test. @janzhongi Would it be difficult to add a test? --- If your project i

[GitHub] incubator-zeppelin pull request: ZEPPELIN-160 Working with provide...

2015-08-21 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/244 ZEPPELIN-160 Working with provided Spark, Hadoop. Zeppelin currently embeds all spark dependencies under interpreter/spark and loading them on runtime. Which is useful becaus

Re: registeredInterpreters and InterpreterFactory.getRegisterInterpreterList return different result

2015-08-21 Thread Amos B. Elberg
I figured it out - if you try to walk the list on setInterpreterGroup, what happens is what I described. I don't know why. As long as you walk the list later it works properly. > On Aug 21, 2015, at 3:59 PM, IT CTO wrote: > > Not sure if it helps your problem but I found the bug here > > publ

[GitHub] incubator-zeppelin pull request: Fix getRegisteredInterpreterList ...

2015-08-21 Thread eranwitkon
Github user eranwitkon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/242#issuecomment-133584373 I noticed this PR failed on CI. based on the log the InterpreterFactory JUnit failed. Not sure what to do with it as the unit test pass locally ![

[GitHub] incubator-zeppelin pull request: Zeppelin-241 Add test for REST AP...

2015-08-21 Thread eranwitkon
GitHub user eranwitkon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/243 Zeppelin-241 Add test for REST API Add tests for all REST API You can merge this pull request into a Git repository by running: $ git pull https://github.com/eranwitkon/incubator

[GitHub] incubator-zeppelin pull request: Configurable origin

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/233 --- 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] incubator-zeppelin pull request: Configurable Origins and small co...

2015-08-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/235 --- 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 fea

[GitHub] incubator-zeppelin pull request: Configurable Origins and small co...

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/235#issuecomment-133566060 Thanks @djoelz. I'm merging 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 pr

[GitHub] incubator-zeppelin pull request: Fix getRegisteredInterpreterList ...

2015-08-21 Thread eranwitkon
GitHub user eranwitkon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/242 Fix getRegisteredInterpreterList to avoid adding null to the list This PR fix the getRegisteredInterpreterList InterpreterFactory is looping through the interpreters class nam

[GitHub] incubator-zeppelin pull request: Add argument check for option and...

2015-08-21 Thread eranwitkon
GitHub user eranwitkon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/241 Add argument check for option and properties to protect from NPE Here is the repro steps: 1) run zeppelin 2) using some REST client call the update settings with the fo

[jira] [Created] (ZEPPELIN-255) createInterpreterGroup cause NPE when called with NULL options or Property

2015-08-21 Thread Eran W (JIRA)
Eran W created ZEPPELIN-255: --- Summary: createInterpreterGroup cause NPE when called with NULL options or Property Key: ZEPPELIN-255 URL: https://issues.apache.org/jira/browse/ZEPPELIN-255 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: Protect from NPE when creating In...

2015-08-21 Thread eranwitkon
Github user eranwitkon closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/223 --- 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] incubator-zeppelin pull request: [Zeppelin-254] getRegisteredInter...

2015-08-21 Thread eranwitkon
Github user eranwitkon closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/240 --- 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] incubator-zeppelin pull request: [Zeppelin-254] getRegisteredInter...

2015-08-21 Thread eranwitkon
GitHub user eranwitkon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/240 [Zeppelin-254] getRegisteredInterpreterList add NULL to the list InterpreterFactory register all interpreters by looping through the config interpreter class list and looking for them

[jira] [Created] (ZEPPELIN-254) getRegisteredInterpreterList add NULL to the list

2015-08-21 Thread Eran W (JIRA)
Eran W created ZEPPELIN-254: --- Summary: getRegisteredInterpreterList add NULL to the list Key: ZEPPELIN-254 URL: https://issues.apache.org/jira/browse/ZEPPELIN-254 Project: Zeppelin Issue Type: Bug

Re: registeredInterpreters and InterpreterFactory.getRegisterInterpreterList return different result

2015-08-21 Thread IT CTO
Not sure if it helps your problem but I found the bug here public List getRegisteredInterpreterList() { List registeredInterpreters = new LinkedList(); for (String className : interpreterClassList) { registeredInterpreters.add(*Interpreter.findRegisteredInterpreterByClassName(className)*

Re: registeredInterpreters and InterpreterFactory.getRegisterInterpreterList return different result

2015-08-21 Thread Amos B. Elberg
Have you checked whether the additional objects can be cast to wrappedinterpreter or classloaderinterpreter? I suspect I'm hitting a similar issue. For the r interpreter, I need to parse the list of interpreters in the group to find the SparkInterpreter. I find a class with that name, but it c

[jira] [Created] (ZEPPELIN-253) EMR Spark deployment: Class com.hadoop.compression.lzo.LzoCodec not found

2015-08-21 Thread Eugene Sapozhnikov (JIRA)
Eugene Sapozhnikov created ZEPPELIN-253: --- Summary: EMR Spark deployment: Class com.hadoop.compression.lzo.LzoCodec not found Key: ZEPPELIN-253 URL: https://issues.apache.org/jira/browse/ZEPPELIN-253

[GitHub] incubator-zeppelin pull request: ZEPPELIN-215: PostgreSQL complete...

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/198#issuecomment-133534082 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 f

[GitHub] incubator-zeppelin pull request: ZEPPELIN-157: Adding Map Visualiz...

2015-08-21 Thread eranwitkon
Github user eranwitkon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-133525197 Did you see my comment from 5 days ago? Now that I can run the code I found this behavior : Try the following code: %sql select Na

[GitHub] incubator-zeppelin pull request: ZEPPELIN-157: Adding Map Visualiz...

2015-08-21 Thread Madhuka
Github user Madhuka commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-133519842 CI Build is test with PR[1], It contain data validations (fully) and leaflet support with map and tutorial with squashing commits [1] https://github.

[GitHub] incubator-zeppelin pull request: Adding leaflet and data validator...

2015-08-21 Thread Madhuka
Github user Madhuka commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/238#issuecomment-133518629 All data validation are added completed. (factories, services, config), Test cases also added `grunt test` out put. ![screenshot from 2015-0

[GitHub] incubator-zeppelin pull request: ZEPPELIN-252: Add PostgreSQL Inte...

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/239#issuecomment-133511724 Looks great!! --- 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] incubator-zeppelin pull request: ZEPPELIN-251: Postgresql Interpre...

2015-08-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/237#issuecomment-133508501 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-252: Add PostgreSQL Inte...

2015-08-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/239#issuecomment-133508427 looks great! --- 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] incubator-zeppelin pull request: Option to choose default interpre...

2015-08-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/222#issuecomment-133507012 @Leemoonsoo great point on copy notebook --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-251: Postgresql Interpre...

2015-08-21 Thread tzolov
Github user tzolov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/237#issuecomment-133493934 ready for reviewing --- 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] incubator-zeppelin pull request: ZEPPELIN-252: Add PostgreSQL Inte...

2015-08-21 Thread tzolov
Github user tzolov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/239#issuecomment-133494039 Ready for reviewing. --- 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] incubator-zeppelin pull request: ZEPPELIN-252: Add PostgreSQL Inte...

2015-08-21 Thread tzolov
GitHub user tzolov opened a pull request: https://github.com/apache/incubator-zeppelin/pull/239 ZEPPELIN-252: Add PostgreSQL Interpreters Documentation to gh-pages You can merge this pull request into a Git repository by running: $ git pull https://github.com/tzolov/incubator-

[GitHub] incubator-zeppelin pull request: ZEPPELIN-247: fix; navbar.control...

2015-08-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/236#issuecomment-133485488 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 f

[GitHub] incubator-zeppelin pull request: Adding leaflet and data validator...

2015-08-21 Thread Madhuka
GitHub user Madhuka opened a pull request: https://github.com/apache/incubator-zeppelin/pull/238 Adding leaflet and data validator - Testing CI Adding leaflet for map Adding mapping for UI Adding map chart for zeppelin adding sample tutorial for notbook on

Re: registeredInterpreters and InterpreterFactory.getRegisterInterpreterList return different result

2015-08-21 Thread IT CTO
Can someone explain how that possible? [image: Selection_018.png] Size == 16 with only 8 values in the List? On Fri, Aug 21, 2015 at 6:26 PM IT CTO wrote: > How come > InterpreterFactory.getRegisterInterpreterList().size() > > https://github.com/apache/incubator-zeppelin/blob/67f7f3e403bdaa84b

registeredInterpreters and InterpreterFactory.getRegisterInterpreterList return different result

2015-08-21 Thread IT CTO
How come InterpreterFactory.getRegisterInterpreterList().size() https://github.com/apache/incubator-zeppelin/blob/67f7f3e403bdaa84b75f72241c418842fcc13696/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java#L313 and Interpreter.registeredInterpreters().size https

[GitHub] incubator-zeppelin pull request: ZEPPELIN-251: Postgresql Interpre...

2015-08-21 Thread tzolov
GitHub user tzolov opened a pull request: https://github.com/apache/incubator-zeppelin/pull/237 ZEPPELIN-251: Postgresql Interpreter README.md documentation Create postgresql README.md documentation You can merge this pull request into a Git repository by running: $ git pull ht

[jira] [Created] (ZEPPELIN-252) Sync the PSQL/README.md with the Zeppelin Interpreters Documentation Page

2015-08-21 Thread Christian Tzolov (JIRA)
Christian Tzolov created ZEPPELIN-252: - Summary: Sync the PSQL/README.md with the Zeppelin Interpreters Documentation Page Key: ZEPPELIN-252 URL: https://issues.apache.org/jira/browse/ZEPPELIN-252

[jira] [Created] (ZEPPELIN-251) Add README.md in /postgresq folder

2015-08-21 Thread Christian Tzolov (JIRA)
Christian Tzolov created ZEPPELIN-251: - Summary: Add README.md in /postgresq folder Key: ZEPPELIN-251 URL: https://issues.apache.org/jira/browse/ZEPPELIN-251 Project: Zeppelin Issue Type:

[jira] [Created] (ZEPPELIN-250) Document PostgreSQL, HAWQ, Greenplum Interpreter

2015-08-21 Thread Christian Tzolov (JIRA)
Christian Tzolov created ZEPPELIN-250: - Summary: Document PostgreSQL, HAWQ, Greenplum Interpreter Key: ZEPPELIN-250 URL: https://issues.apache.org/jira/browse/ZEPPELIN-250 Project: Zeppelin

JSON response issue

2015-08-21 Thread IT CTO
I am trying to write more test for the REST API but... Does anyone know why we are not getting just the body when we call gson.fromJson(get.getResponseBodyAsString(), new TypeToken>(){}.getType()); get.getResponseBodyAsString() returns *{"status":"OK","message":"","body":[{"id":"2AXE167H3","nam

[GitHub] incubator-zeppelin pull request: feat: use ZEPPELIN_HOSTNAME to sp...

2015-08-21 Thread jonbuffington
Github user jonbuffington commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/221#issuecomment-133364295 I am closing this in favor of #235. --- 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: feat: use ZEPPELIN_HOSTNAME to sp...

2015-08-21 Thread jonbuffington
Github user jonbuffington closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/221 --- 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: Configurable Origins and small co...

2015-08-21 Thread jonbuffington
Github user jonbuffington commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/235#issuecomment-133364163 @djoelz Thanks for adding the origin names a configuration option. Much appreciated. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: ZEPPELIN-215: PostgreSQL complete...

2015-08-21 Thread tzolov
Github user tzolov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/198#issuecomment-133362004 This is ready to be reviewed and merged. > Note that the underlying design uses pure JDBC API and will work for any JDBC driver. Here are some o

Re: Improving Visualization of zeppelin with GSOC

2015-08-21 Thread madhuka udantha
Hi, Guillaume Currently It is showing Bounds and Markers, Since leaflet support trajectories, line, path, GeoJSON etc. We will able added them for zeppelin map. First we will added current PR to zeppelin master and will able to work on extending it for those features. Thanks -- Cheers, Madhuk

[GitHub] incubator-zeppelin pull request: Added Shiro security

2015-08-21 Thread hayssams
Github user hayssams commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/53#issuecomment-133314006 Hello Just came back from vacation. Give me a couple of weeks to build the required tests. Cordialement. *Hayssam Saleh, Architecture

Re: Improving Visualization of zeppelin with GSOC

2015-08-21 Thread tog
Hi madhuka Thanks for your work, I am very interested by using the map visualization and possibly extend to visualize (time dependent) trajectories. Any idea when this will become available in master? Cheers Guillaume Hi All, You can find all the tasks and milestones on GSOC 2015 in Zeppelin in

[GitHub] incubator-zeppelin pull request: ZEPPELIN-247: fix; navbar.control...

2015-08-21 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/236 ZEPPELIN-247: fix; navbar.controller.js used to assumes WebSocket connection as disconnected. ZEPPELIN-247: fix; navbar.controller.js used to assumes WebSocket connection as disc