You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by blrunner <gi...@git.apache.org> on 2015/11/16 18:30:51 UTC
[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...
GitHub user blrunner opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/439
ZEPPELIN-427: Upgrade to Apache TAJO 0.11.0
I bumped up Tajo to 0.11.0. And I enabled ```cancel()``` because Tajo supports it since 0.11.0 release. For the reference, this patch ran successfully on my Tajo testing cluster.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/blrunner/incubator-zeppelin ZEPPELIN-427
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/439.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 #439
----
commit 2b81d4f1f173bcdc48240fe347393b123b56ee07
Author: JaeHwa Jung <bl...@apache.org>
Date: 2015-11-16T17:26:18Z
Bump up Tajo to 0.11.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-427: Upgrade to Apache T...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/439
---
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-427: Upgrade to Apache T...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157278977
I think the last CI fail is irrelevant.
Looks good to me.
---
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-427: Upgrade to Apache T...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157246146
I made two CI builds and one of them succeeded as following:
https://travis-ci.org/apache/incubator-zeppelin/builds/91511236
---
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-427: Upgrade to Apache T...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157233336
Hi @Leemoonsoo
Thank you for your quick feedback. I've just added some unit test cases. The CI build will run soon. :)
---
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-427: Upgrade to Apache T...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157386916
Merge if there're no more discussions
---
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-427: Upgrade to Apache T...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157184867
Thanks for nice upgrade.
CI looks like failing with the other reason, but could you try trigger CI? close / reopen this PR will trigger the CI.
---
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-427: Upgrade to Apache T...
Posted by blrunner <gi...@git.apache.org>.
Github user blrunner commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157547361
There seems to be no problem. Could you merge this PR? :-)
---
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.
---