[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-15 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-184131164 Cool. Meanwhile I have added documentation around using Drill with JDBC Interpreter via PR #718 --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-10 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-182393653 Agreed. I had this in my list of TODO. Will take some time out over weekend & try to add this information. --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-10 Thread smartinsightsfromdata
Github user smartinsightsfromdata commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-182387053 @jongyoul @malur I think the relevance of Apache Drill deserves at the very least a document with a dedicated set of instructions on how to

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-10 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-182452023 @smartinsightsfromdata @malur I also have a plan to enable customization for specific jdbc driver, for example, completers and dedicated word and so on.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-181716616 @tmx11 Hi, could you please those jars under `/interpreters/jdbc/`? I think this would be more general position to use that interpreter. If it still

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-27 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-175933833 Closing this PR, as generic JDBC provides the needed functionality. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-27 Thread malur
Github user malur closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/644 --- 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-576] Support Apache Dri...

2016-01-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-174817516 @malur Thanks for positive answer. And could you please close this PR? --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-20 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-173449722 @jongyoul Sure. I will check the latest code with Drill & Update docs, if needed. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-19 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-173119037 @malur Sorry for late reply. In my opinion, JDBCInterpreter can cover this feature. If it's not, please let me know. Otherwise, how about updating docs for

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-17 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-172320004 Hi @malur, I think it's a good idea to wait to be done merge the Generic JDBC and work on that to add support for everything related to JDBC

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-17 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-172320196 @vgmartinez, that would be right approach. I will wait for the merge to happen. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-17 Thread malur
Github user malur commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-172318193 I looked into the Generic JDBC approach. Looks good. So, * We need to use the Generic JDBC Interpreter code & extend that to provide specific

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-01-15 Thread malur
GitHub user malur opened a pull request: https://github.com/apache/incubator-zeppelin/pull/644 [ZEPPELIN-576] Support Apache Drill Interpreter ### What is this PR for? Supports Apache Drill Interpreter ### What type of PR is it? Improvement, Feature ###