[GitHub] incubator-quickstep pull request #319: Fixed the bug when partition w/ prune...

[zuyu <gi...@git.apache.org>]

GitHub user zuyu opened a pull request:

    https://github.com/apache/incubator-quickstep/pull/319

    Fixed the bug when partition w/ pruned columns.

    This PR fixed the bug that when some column gets pruned, the partition is incorrect due to the wrong logical catalog attribute id from the output relation.
    
    We should use the old logical catalog attribute id from *the input relation*.

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

    $ git pull https://github.com/zuyu/incubator-quickstep fix-pruned-col-w-partitions

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

    https://github.com/apache/incubator-quickstep/pull/319.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 #319
    
----
commit 8b0b05ca259ed552917a8a9bcc7fed7d28d541ea
Author: Zuyu Zhang <zu...@cs.wisc.edu>
Date:   2017-10-24T21:17:50Z

    Fixed the bug when partition w/ pruned columns.

----


---

[GitHub] incubator-quickstep pull request #319: Fixed the bug when partition w/ prune...

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-quickstep/pull/319


---

[GitHub] incubator-quickstep issue #319: Fixed the bug when partition w/ pruned colum...

[jianqiao <gi...@git.apache.org>]

Github user jianqiao commented on the issue:

    https://github.com/apache/incubator-quickstep/pull/319
  
    LGTM! Merging.


---