[GitHub] zeppelin issue #2396: [ZEPPELIN-2451]: Add JDBC config option for calling co...

2017-12-28 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2396 @jongyoul I fixed the Appveyor test, but having problems with the Jenkins Python tests. I don't believe these are related to my changes.. can you suggest what I can try next? ---

[GitHub] zeppelin issue #2396: [ZEPPELIN-2451]: Add JDBC config option for calling co...

2017-12-12 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2396 @jongyoul I'm having trouble with CI. Jenkins tests pass except 3 that relate to Python interpreter, I believe unrelated to these small JDBC interpreter changes. Are the Python

[GitHub] zeppelin pull request #2396: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-12-12 Thread randerzander
GitHub user randerzander reopened a pull request: https://github.com/apache/zeppelin/pull/2396 [ZEPPELIN-2451]: Add JDBC config option for calling connection.commit after paragraph execution Updated version of: https://github.com/apache/zeppelin/pull/2284 ### What is this

[GitHub] zeppelin pull request #2396: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-12-12 Thread randerzander
Github user randerzander closed the pull request at: https://github.com/apache/zeppelin/pull/2396 ---

[GitHub] zeppelin pull request #2396: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-06-07 Thread randerzander
GitHub user randerzander reopened a pull request: https://github.com/apache/zeppelin/pull/2396 [ZEPPELIN-2451]: Add JDBC config option for calling connection.commit after paragraph execution Updated version of: https://github.com/apache/zeppelin/pull/2284 ### What is this

[GitHub] zeppelin pull request #2396: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-06-07 Thread randerzander
Github user randerzander closed the pull request at: https://github.com/apache/zeppelin/pull/2396 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #2396: [ZEPPELIN-2451]: Add JDBC config option for calling co...

2017-06-07 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2396 retest --- 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 #2284: [ZEPPELIN-2451]: Add JDBC config option for calling co...

2017-06-05 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2284 Closed in favor of https://github.com/apache/zeppelin/pull/2396 --- 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 pull request #2396: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-06-05 Thread randerzander
GitHub user randerzander opened a pull request: https://github.com/apache/zeppelin/pull/2396 [ZEPPELIN-2451]: Add JDBC config option for calling connection.commit after paragraph execution Updated version of: https://github.com/apache/zeppelin/pull/2284 ### What is this PR

[GitHub] zeppelin pull request #2284: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-06-05 Thread randerzander
Github user randerzander closed the pull request at: https://github.com/apache/zeppelin/pull/2284 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #2284: [ZEPPELIN-2451]: Add JDBC config option for calling co...

2017-05-05 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2284 Looks like recent commits conflict with this fix. I'll re-synch with master, but @Leemoonsoo any comments on the approach before I do that? --- If your project is set up for it, yo

[GitHub] zeppelin pull request #2284: [ZEPPELIN-2451]: Add JDBC config option for cal...

2017-04-25 Thread randerzander
GitHub user randerzander opened a pull request: https://github.com/apache/zeppelin/pull/2284 [ZEPPELIN-2451]: Add JDBC config option for calling connection.commit after paragraph execution ### What is this PR for? Adding config for automatically calling commit after JDBC

[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-10 Thread randerzander
GitHub user randerzander reopened a pull request: https://github.com/apache/zeppelin/pull/2117 ZEPPELIN-2241: JDBC interpreter throws npe on connecting to any db that has a schema with "null" name ### What is this PR for? A few sentences describing the overall goals o

[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-10 Thread randerzander
Github user randerzander closed the pull request at: https://github.com/apache/zeppelin/pull/2117 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-10 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2117 @1ambda I enabled Travis for my repo. All builds passed except [this one](https://travis-ci.org/randerzander/zeppelin/jobs/209828896) which unrelated errors about Pig/MapReduce. --- If

[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-10 Thread randerzander
Github user randerzander closed the pull request at: https://github.com/apache/zeppelin/pull/2117 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-10 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/2117 Thanks for the link, @1ambda . Do I need to close and re-open the PR to get Travis to run the test now? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-10 Thread randerzander
GitHub user randerzander reopened a pull request: https://github.com/apache/zeppelin/pull/2117 ZEPPELIN-2241: JDBC interpreter throws npe on connecting to any db that has a schema with "null" name ### What is this PR for? A few sentences describing the overall goals o

[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-09 Thread randerzander
GitHub user randerzander opened a pull request: https://github.com/apache/zeppelin/pull/2117 ZEPPELIN-2241: JDBC interpreter throws npe on connecting to any db that has a schema with "null" name ### What is this PR for? A few sentences describing the overall goals o

[GitHub] zeppelin issue #1617: ZEPPELIN-1645: JDBC Interpreter does not commit update...

2016-11-24 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1617 The failed test is in the Python interpreter, unrelated to this JDBC change. --- 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 #1617: ZEPPELIN-1645: JDBC Interpreter does not commit update...

2016-11-24 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1617 @astroshim Added a safety check on autocommit status --- 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 #1617: ZEPPELIN-1645: JDBC Interpreter does not commit update...

2016-11-08 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1617 The failed test is some bower.json issue, unrelated to the change in 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

[GitHub] zeppelin pull request #1617: ZEPPELIN-1645: JDBC Interpreter does not commit...

2016-11-08 Thread randerzander
GitHub user randerzander opened a pull request: https://github.com/apache/zeppelin/pull/1617 ZEPPELIN-1645: JDBC Interpreter does not commit updates ### What is this PR for? Allow users to issue insert/update/upsert statements from Zeppelin notes ### What type of PR is

[GitHub] zeppelin issue #728: Leaflet support, based on PR #152

2016-09-25 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/728 @eddies what CSS tricks did you have to do to get GeoJson to work with Leaflet in Zeppelin? I'm trying to follow the simple example here http://leafletjs.com/examples/geojson

[GitHub] zeppelin issue #1442: ZEPPELIN-1459: Zeppelin JDBC URL properties mangled

2016-09-20 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1442 I tested the PR and it works as expected. Thanks @prabhjyotsingh! --- 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 #1436: ZEPPELIN-1452: Include Phoenix 'thin client' instructi...

2016-09-19 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1436 Before merging and implying support for Phoenix's thin client, [ZEPPELIN-1459](https://issues.apache.org/jira/browse/ZEPPELIN-1459) should also be resolved. --- If your project is s

[GitHub] zeppelin issue #1437: ZEPPELIN-1454: Wrong property value on interpreter pag...

2016-09-19 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1437 I believe the test failure is unrelated. However, before merging this and implying support for the thin client, I ran into another issue with mangled JDBC properties in the URL

[GitHub] zeppelin pull request #1436: Added Phoenix 'thin client' to JDBC docs

2016-09-18 Thread randerzander
GitHub user randerzander opened a pull request: https://github.com/apache/zeppelin/pull/1436 Added Phoenix 'thin client' to JDBC docs ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out t

[GitHub] zeppelin issue #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1048 I tried again from your branch and it worked. I must have applied your commit incorrectly last time.. my bad! Thanks for the fix =) --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1048 @astroshim still getting same errors. After trying to "move up" and then "delete" a cell, I get these errors in the log: http://pastebin.com/Sr4WveXF --- If your proj

[GitHub] zeppelin issue #1034: [ZEPPELIN-1009] [HOTFIX] Fix Selenium test error

2016-06-19 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1034 @bzz @Leemoonsoo, you said "i believe it locates Inserts New button and click(), but nothing happens and test fails". This may be related to https://issues.apache.org/j