You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by malur <gi...@git.apache.org> on 2016/02/15 10:25:58 UTC
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
GitHub user malur opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/718
Added details on using Drill with JDBC Interpreter
## What is this PR for?
Documentation to configure Drill using JDBC Interpreter
## What type of PR is it?
Documentation
## Todos
## Is there a relevant Jira issue?
No. But there is related PR: https://issues.apache.org/jira/browse/ZEPPELIN-614
## How should this be tested?
## Screenshots (if appropriate)
## Questions:
Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/walmart/incubator-zeppelin drill-documentation
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/718.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #718
----
commit 4b6f82523f4d72af82aa59424ef993c367b0cd33
Author: Ramu Malur <ra...@walmart.com>
Date: 2016-02-15T09:21:16Z
Added details on using Drill with JDBC Interpreter
----
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185453034
Merging if there's no more discussion
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184756035
@malur yes adding drill to the supported DB in the JDBC documentation.
However most of the content of your document is already inside the JDBC, so I doubt there is a need for a separate document
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184262845
@malur
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184266001
@malur If you want to find this docs in Zeppelin documentation web page, you have to add `<li><a href="{{BASE_PATH}}/interpreter/drill.html">Drill</a></li>` below [this line](https://github.com/apache/incubator-zeppelin/blob/master/docs/_includes/themes/zeppelin/_navigation.html#L41).
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/718
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by malur <gi...@git.apache.org>.
Github user malur commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185058046
@corneadoug Sounds good.
I saw specific files for different DBs thats supported by JDBC Interpreter & added one for drill. I have deleted drill.md & updated jdbc.md accordingly.
Also, the step, informing the placement of jars in a folder & updating CLASSPATH, for configuring other drivers doesn't seem to work.
What worked for me is placing the jars under ZEPPELIN_HOME/interpreter/jdbc/. Should I raise JIRA or am I missing anything?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by malur <gi...@git.apache.org>.
Github user malur commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185105640
@minahlee Awesome!
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by anthonycorbacho <gi...@git.apache.org>.
Github user anthonycorbacho commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185455520
:+1:
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185295506
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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184509145
@malur Thanks for the documentation, but how about improving JDBC document? It looks good by itself, but basically, because drill is using JDBC interpreter, It looks better to improve the jdbc.md. How about you?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by malur <gi...@git.apache.org>.
Github user malur commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184510375
@jongyoul Are you talking about adding sections in jdbc.md for configuring different Databases? Wouldn't the JDBC page be too lengthy, if we add all in one page ? How about adding Drill to the Supported Databases section & provide a link to this drill.md from there?
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Added details on using Drill with...
Posted by minahlee <gi...@git.apache.org>.
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-185070059
@malur I recently made a PR for updating Zeppelin classpath via GUI for different JDBC drivers. Please see #717
---
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---