[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-25 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2396 @randerzander autocommit is already in JDBCInterpreter. ``` finally { //In case user ran an insert/update/upsert statement if (connection != null) {

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

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

2017-12-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2396 You'd better rebase from current master to make your branch newer. ---

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

2017-11-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2396 @randerzander Can you fix the failure of CI? ---

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

2017-11-27 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2396 LGTM ---

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

2017-11-18 Thread tzolov
Github user tzolov commented on the issue: https://github.com/apache/zeppelin/pull/2396 The `JDBC interpreter` uses Connection's auto-commit status to decide whether forceful to commit the connection. As it as pointed above `commit` is not supported by many `NoSql` Jdbc

[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