You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by jongyoul <gi...@git.apache.org> on 2016/06/01 06:58:01 UTC
[GitHub] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...
GitHub user jongyoul opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/943
ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - https://github.com/apache/incubator-zeppelin/blob/master/CONTRIBUTING.md
### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]
### Todos
* [ ] - Task
### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533]
### How should this be tested?
Outline the steps to test the PR here.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jongyoul/incubator-zeppelin ZEPPELIN-925
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/943.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 #943
----
commit c983432b4ec87a87c78b0417760035481f16c718
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:35:04Z
Update example for hive in jdbc.md
commit f5e613a3d6dd8dd6aa251a0a17a5b183648a074f
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:01Z
Remove hive module and its directory
commit e6822b6dd5cf27f51a02e56d7216166539ed0a24
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:32Z
Remove hive.md from docs/interpreter
commit 7f85e15770952f7101c9e084e606d7d461de7757
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:56:38Z
Updated Documentation
commit 6f7b1fe17c516c771102fd8377ba5dd3c6fd84d1
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:57:18Z
Deleted hive module from pom.xml
----
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@felixcheung @bzz @minahlee @AhyoungRyu I've reverted hive.md and updated documentation. Please check it finally.
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
Merging 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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/943
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@jongyoul Yeah looks good :)
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by vgmartinez <gi...@git.apache.org>.
Github user vgmartinez commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@jongyoul ok ok, all external dependencies loaded them that way. I have been a time out...;)
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@bzz @minahlee Please check it finally. And I email {dev,user}@ to announce this change.
---
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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/943
re-trigger
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
re-trigger
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by vgmartinez <gi...@git.apache.org>.
Github user vgmartinez commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
Hi @jongyoul, great job with all integrations in the jdbc interpreter.
What do you think, about putting hive dependence into the pom and so it is not necessary add the jar in the classpath.
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@bzz I made a https://issues.apache.org/jira/browse/ZEPPELIN-936
---
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 #943: ZEPPELIN-925 Merge HiveInterpreter int...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/943
---
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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by bzz <gi...@git.apache.org>.
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/943
CI is failing by this test, and I do not think we have seen it failing before.
```
Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 168.646 sec <<< FAILURE! - in org.apache.zeppelin.rest.ZeppelinSparkClusterTest
sparkRTest(org.apache.zeppelin.rest.ZeppelinSparkClusterTest) Time elapsed: 11.03 sec <<< FAILURE!
java.lang.AssertionError: expected:<FINISHED> but was:<ERROR>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:743)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at org.apache.zeppelin.rest.ZeppelinSparkClusterTest.sparkRTest(ZeppelinSparkClusterTest.java:104)
```
---
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 #943: ZEPPELIN-925 Merge HiveInterpreter int...
Posted by jongyoul <gi...@git.apache.org>.
GitHub user jongyoul reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/943
ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter
### What is this PR for?
This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using loading dynamic dependencies. It reduces Zeppelin's binary size.
There's no codes' modification except removing hive directory and remove the module from pom.xml
### What type of PR is it?
[Feature]
### Todos
* [x] - Remove hive module
* [x] - Add an example for using Hive in JDBC
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-925
### How should this be tested?
Set the interpreter properties and test it
### 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/jongyoul/incubator-zeppelin ZEPPELIN-925
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/943.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 #943
----
commit 8db6b21cf1ca44291531e47a3dc436ee7213ce66
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:35:04Z
Update example for hive in jdbc.md
commit 3a6a26c9c0a26bd6ba5b95eedd9d81f2a285460c
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:01Z
Remove hive module and its directory
commit d00aa6c068401bbf83528f9c3f31319d51912c8a
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:32Z
Remove hive.md from docs/interpreter
commit 333c8800609eb36157964e6d2339243ea130d0d4
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:56:38Z
Updated Documentation
commit 95bd5ef7bc672e7586903b4ed394c57f99158029
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:57:18Z
Deleted hive module from pom.xml
commit 9335ab03cb6bfa6245fad55b0307f002e7a77031
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-02T02:47:29Z
Revert "Remove hive.md from docs/interpreter"
This reverts commit e6822b6dd5cf27f51a02e56d7216166539ed0a24.
commit b8821f9bfe0b1fb3e36032707dcb3d3ff5aa0f3c
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-07T06:35:44Z
Update hive.md
commit b8866a3186397411e2955e47f1725d2645b4e43e
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-07T06:42:03Z
Updated style of jdbc.md
commit 3fbe9811206cff8851ee15785129b80ac10904b8
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-07T06:43:46Z
Fixed typo
----
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
Update documentation and rebase it from current master
---
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 #943: ZEPPELIN-925 Merge HiveInterpreter int...
Posted by jongyoul <gi...@git.apache.org>.
GitHub user jongyoul reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/943
ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter
### What is this PR for?
This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using loading dynamic dependencies. It reduces Zeppelin's binary size.
There's no codes' modification except removing hive directory and remove the module from pom.xml
### What type of PR is it?
[Feature]
### Todos
* [x] - Remove hive module
* [x] - Add an example for using Hive in JDBC
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-925
### How should this be tested?
Set the interpreter properties and test it
### 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/jongyoul/incubator-zeppelin ZEPPELIN-925
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/943.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 #943
----
commit c983432b4ec87a87c78b0417760035481f16c718
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:35:04Z
Update example for hive in jdbc.md
commit f5e613a3d6dd8dd6aa251a0a17a5b183648a074f
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:01Z
Remove hive module and its directory
commit e6822b6dd5cf27f51a02e56d7216166539ed0a24
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:32Z
Remove hive.md from docs/interpreter
commit 7f85e15770952f7101c9e084e606d7d461de7757
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:56:38Z
Updated Documentation
commit 6f7b1fe17c516c771102fd8377ba5dd3c6fd84d1
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:57:18Z
Deleted hive module from pom.xml
commit e6c5340ea1ff495f590e76e4fd0ed0a56eb4d192
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-02T02:47:29Z
Revert "Remove hive.md from docs/interpreter"
This reverts commit e6822b6dd5cf27f51a02e56d7216166539ed0a24.
commit afd56de9bbfbd4ebb63c1bdf5f8ce9b78dc6c366
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-02T02:54:33Z
Update documentation to guide a JDBC Interpreter
commit cb03fe1862cf318c645c4aeaf6822e3e56f0a560
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-02T06:07:41Z
Update example in hive.md
commit 684b382422ce42ab4a7b0fd13155024d52e03b5a
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-02T06:37:55Z
Fixed documentation
----
---
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 #943: ZEPPELIN-925 Merge HiveInterpreter int...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/943
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by minahlee <gi...@git.apache.org>.
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@jongyoul changes looks good to me. About the documentation I think same as @bzz for the users who were already using hive interpreter. I think it would be good to keep hive.md document and mention that hive interpreter is merged into master and refer the jdbc interpreter doc link.
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@felixcheung How about it?
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@vgmartinez Now, Zeppelin enable load dependencies dynamically, thus if you set dependencies in your interpreter tab, it works. Thus, we don't have to include any dependencies any more. See an example below:
<img width="1054" alt="screen shot 2016-06-08 at 3 00 54 pm" src="https://cloud.githubusercontent.com/assets/3612566/15886295/30e21efe-2d97-11e6-9842-8d269d7bd867.png">
---
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 #943: ZEPPELIN-925 Merge HiveInterpreter int...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/943
---
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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/943
That failure is irrelevant to this PR.
```
Results :
Failed tests:
ZeppelinSparkClusterTest.pySparkDepLoaderTest:231 expected:<FINISHED> but was:<ERROR>
Tests run: 39, Failures: 1, Errors: 0, Skipped: 0
```
---
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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/943
@minahlee Could you please review this PR, especially setting dependencies?
---
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 issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@bzz What you have seen is different from the first error. Have you ever seen the first one before? I'll make a new issue for second error with `flaky-test` tag.
And it's good to remain documentation about hive until - may be - 0.7.0 to guide that users use a JDBC interpreter to use hive. I'll update hive.md again.
---
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: ZEPPELIN-925 Merge HiveInterpreter into JDB...
Posted by jongyoul <gi...@git.apache.org>.
GitHub user jongyoul reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/943
ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter
### What is this PR for?
This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using loading dynamic dependencies. It reduces Zeppelin's binary size.
There's no codes' modification except removing hive directory and remove the module from pom.xml
### What type of PR is it?
[Feature]
### Todos
* [x] - Remove hive module
* [x] - Add an example for using Hive in JDBC
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-925
### How should this be tested?
Set the interpreter properties and test it
### 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/jongyoul/incubator-zeppelin ZEPPELIN-925
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/943.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 #943
----
commit c983432b4ec87a87c78b0417760035481f16c718
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:35:04Z
Update example for hive in jdbc.md
commit f5e613a3d6dd8dd6aa251a0a17a5b183648a074f
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:01Z
Remove hive module and its directory
commit e6822b6dd5cf27f51a02e56d7216166539ed0a24
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:42:32Z
Remove hive.md from docs/interpreter
commit 7f85e15770952f7101c9e084e606d7d461de7757
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:56:38Z
Updated Documentation
commit 6f7b1fe17c516c771102fd8377ba5dd3c6fd84d1
Author: Jongyoul Lee <jo...@gmail.com>
Date: 2016-06-01T06:57:18Z
Deleted hive module from pom.xml
----
---
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.
---