You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by jinossy <gi...@git.apache.org> on 2015/02/10 08:18:07 UTC
[GitHub] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
GitHub user jinossy opened a pull request:
https://github.com/apache/tajo/pull/380
TAJO-1160: Remove Hadoop dependency from tajo-client module.
I’ve refer to the thinClient of hyunsik repo
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jinossy/tajo TAJO-1160
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tajo/pull/380.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 #380
----
commit 8b9f45d9e2f81e105b7be75ad85fa710d840b42a
Author: jhkim <jh...@apache.org>
Date: 2015-02-10T07:13:52Z
TAJO-1160: Remove Hadoop dependency from tajo-client module.
----
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by jinossy <gi...@git.apache.org>.
Github user jinossy commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74009231
Thank you for the review
A simple query are not to allocate a query master. I've fix the bug.
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/tajo/pull/380
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74216405
Hi @jinossy
+1
Thanks for your great works. Tajo jdbc file looks pretty to me.
And it integrated with JDBC support tools, such as SQL workbench, DbVisualizer.
If TravisCI build succeed, push it. :)
Cheers
Jaehwa
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by hyunsik <gi...@git.apache.org>.
Github user hyunsik commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74213529
@jinossy I just reviewed your patch. The patch looks nice to me. BTW, could you trigger TravisCI?
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74005407
Hi @jinossy
Thanks for your works.
But I found a build failure as follows:
{code:xml}
Results :
Tests in error:
TestTajoClient.testGetQueryStatusAndResultAfterFinish:644 » Service org.apache...
TestTajoClient.testGetFinishedQueryList:617 » Service org.apache.tajo.rpc.Tajo...
Tests run: 1361, Failures: 0, Errors: 2, Skipped: 0
{code}
Could you check it 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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by jinossy <gi...@git.apache.org>.
Github user jinossy commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74250520
TravisCI failure is network issue of maven repository.
I've just trigger the TravisCI and I'll commit it after CI is succeeded.
---
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] tajo pull request: TAJO-1160: Remove Hadoop dependency from tajo-c...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/tajo/pull/380#issuecomment-74217393
For reference, all unit test cases finished successfully on my laptop.
---
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.
---