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

2017-03-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2117 LGTM and merge to master and branch-0.7, if no further comments. --- 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 #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 LGTM except one minor thing. --- 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 #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-11 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 I will test and give you feed back 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 not have this feature

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

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 @randerzander Hi, you can restart the failed job again using the `restart` button. If you close / open this PR after all jobs pass, you can see this

[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 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