You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by khalidhuseynov <gi...@git.apache.org> on 2015/05/12 09:07:42 UTC

[GitHub] incubator-zeppelin pull request: Tajo jdbc thrift interpreters

GitHub user khalidhuseynov opened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/73

    Tajo jdbc thrift interpreters

    This is for decoupling Tajo-jdbc and Tajo-thrift interpreters. 
    
    TODO
    - [x] Rename Tajo interpreter explicitly into tajo-jdbc
    - [x] Add tajo-thrift interpreter
    - [x] Implement **getProgress()** for tajo-thrift interpreter, by @bzz 
    - [x] Learning Zeppelin architecture and handling unexpected problems 
    - [ ] Implement **cancel()**


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/khalidhuseynov/incubator-zeppelin tajo-jdbc-thrift-interpreters

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/73.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 #73
    
----
commit 7ba4697e02875064a122be4a4c89e63cf6cd20d3
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:14:00Z

    change package from tajo to tajo.jdbc in config files

commit dc4f0fadde78013050a68b432b05408f39f36d6d
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:23:42Z

    change path (renaming)  from tajo to tajo.jdbc package

commit 4d1dd5b2cf190e9889c81b1ec5b0284317026484
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:28:12Z

    config changes for tajo-thrift interpreter

commit afd305fed7d2f929d896a6b61629d8d2f1369c48
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:36:19Z

    add Tajo-thrift interpreter

commit 8dd6ebd1c51849c292bae71926149414e083b0e6
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:37:12Z

    getProgress() implementation

----


---
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: Tajo jdbc thrift interpreters

Posted by khalidhuseynov <gi...@git.apache.org>.
Github user khalidhuseynov commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/73#issuecomment-101554581
  
    Triggering CI Build 


---
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: Tajo jdbc thrift interpreters

Posted by khalidhuseynov <gi...@git.apache.org>.
Github user khalidhuseynov closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/73


---
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: Tajo jdbc thrift interpreters

Posted by khalidhuseynov <gi...@git.apache.org>.
GitHub user khalidhuseynov reopened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/73

    Tajo jdbc thrift interpreters

    This is for decoupling Tajo-jdbc and Tajo-thrift interpreters. 
    
    TODO
    - [x] Rename Tajo interpreter explicitly into tajo-jdbc
    - [x] Add tajo-thrift interpreter
    - [x] Implement **getProgress()** for tajo-thrift interpreter, by @bzz 
    - [x] Learning Zeppelin architecture and handling unexpected problems 
    - [ ] Implement **cancel()**


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/khalidhuseynov/incubator-zeppelin tajo-jdbc-thrift-interpreters

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/73.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 #73
    
----
commit 7ba4697e02875064a122be4a4c89e63cf6cd20d3
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:14:00Z

    change package from tajo to tajo.jdbc in config files

commit dc4f0fadde78013050a68b432b05408f39f36d6d
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:23:42Z

    change path (renaming)  from tajo to tajo.jdbc package

commit 4d1dd5b2cf190e9889c81b1ec5b0284317026484
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:28:12Z

    config changes for tajo-thrift interpreter

commit afd305fed7d2f929d896a6b61629d8d2f1369c48
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:36:19Z

    add Tajo-thrift interpreter

commit 8dd6ebd1c51849c292bae71926149414e083b0e6
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-12T06:37:12Z

    getProgress() implementation

commit 7d9b9734c8cfe7cbb62272fdca12bc8f8bd2f152
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-13T02:05:20Z

    add -incubation label to version id in pom.xml

commit 77632ea563395fa0844fb35c8b1c0a9a57072043
Author: Khalid Huseynov <kh...@nflabs.com>
Date:   2015-05-13T07:27:18Z

    add temporary reference to tajo-thrift-server repo

----


---
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: Tajo jdbc thrift interpreters

Posted by bzz <gi...@git.apache.org>.
Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/73#issuecomment-101475806
  
    @khalidhuseynov build fails, can you fix it please?


---
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: Tajo jdbc thrift interpreters

Posted by khalidhuseynov <gi...@git.apache.org>.
Github user khalidhuseynov closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/73


---
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: Tajo jdbc thrift interpreters

Posted by bzz <gi...@git.apache.org>.
Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/73#issuecomment-143169281
  
    Shall we update or close this one, after #39 ?


---
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.
---