You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hawq.apache.org by vraghavan78 <gi...@git.apache.org> on 2016/05/11 03:46:05 UTC

[GitHub] incubator-hawq pull request: Remove CTranslatorPlStmtToDXL Deadcod...

GitHub user vraghavan78 opened a pull request:

    https://github.com/apache/incubator-hawq/pull/646

    Remove CTranslatorPlStmtToDXL Deadcode [#119102697]

    Cleaning dead code. [#119102697]
    
    Context: When we did not have optimization modules in Orca and just DXL, we used CTranslatorPlStmtToDXL to test correctness of translation of DXL to PlStmt.
    
    @changleicn @liming01 please review and merge this PR.

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

    $ git pull https://github.com/vraghavan78/incubator-hawq deadcode

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

    https://github.com/apache/incubator-hawq/pull/646.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 #646
    
----
commit e7a85a42d99064c251197952307c10eee5a6b2b1
Author: Venkatesh (Venky) Raghavan <vr...@pivotal.io>
Date:   2016-05-11T03:43:13Z

    Remove CTranslatorPlStmtToDXL Deadcode [#119102697]

----


---
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-hawq pull request: Remove CTranslatorPlStmtToDXL Deadcod...

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

    https://github.com/apache/incubator-hawq/pull/646#issuecomment-222340666
  
    HAWQ JIRA issue was created for this PR: https://issues.apache.org/jira/browse/HAWQ-764
    @vraghavan78 Please update 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.
---

[GitHub] incubator-hawq pull request #646: Remove CTranslatorPlStmtToDXL Deadcode [#1...

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

    https://github.com/apache/incubator-hawq/pull/646


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