[jira] [Commented] (HAWQ-79) Re-enable unit test for shared input scan

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HAWQ-79?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14987812#comment-14987812 ] 

ASF GitHub Bot commented on HAWQ-79:
------------------------------------

GitHub user armenatzoglou opened a pull request:

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

    HAWQ-79, HAWQ-85. Re-enable unit test for executor

    Executor's unit tests were not enabled. Previous commits had renamed executor/test folder to test_discard. I renamed the folder back to test and edited Makefile and nodeSubplan_test files. 
    
    The tests were red, because the code was not up-to-date, e.g., Makefile was including files that do not exist anymore.

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

    $ git pull https://github.com/gcaragea/incubator-hawq nikos-HAWQ-79

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

    https://github.com/apache/incubator-hawq/pull/72.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 #72
    
----
commit 7fa570ed1abc221360c8d961846b112867a7c455
Author: Nikos Armenatzoglou <ni...@gmail.com>
Date:   2015-11-02T21:57:38Z

    HAWQ-79. Re-enable unit test for shared input scan

----


> Re-enable unit test for shared input scan
> -----------------------------------------
>
>                 Key: HAWQ-79
>                 URL: https://issues.apache.org/jira/browse/HAWQ-79
>             Project: Apache HAWQ
>          Issue Type: Test
>            Reporter: Goden Yao
>
> shared input scan unit test currently is disabled:
> 1. src/backend/executor/test_discard/nodeShareInputScan_test.c
> 2. src/backend/executor/test_discard/execAmi_test.c
> To enable them, we need to move the test to "src/test" structure, not in "src/backend"



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)