You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@quickstep.apache.org by zuyu <gi...@git.apache.org> on 2017/09/29 17:14:41 UTC

[GitHub] incubator-quickstep pull request #304: Added a new set API for TupleIdSequen...

GitHub user zuyu opened a pull request:

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

    Added a new set API for TupleIdSequence.

    This PR added a new `set` API for `TupleIdSequence`, and has some performance improvement.
    
    Assigned to @jianqiao.
    
    ![chart-3](https://user-images.githubusercontent.com/2245572/31027397-b7e2c836-a50f-11e7-9910-bdc8f1f1e825.png)


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

    $ git pull https://github.com/zuyu/incubator-quickstep simplified-set-tuple-id

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

    https://github.com/apache/incubator-quickstep/pull/304.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 #304
    
----
commit 82341835aaeb05525dd7822935bd0729726d54c0
Author: Zuyu Zhang <zu...@cs.wisc.edu>
Date:   2017-09-29T00:28:30Z

    Added a new set API for TupleIdSequence.

----


---

[GitHub] incubator-quickstep issue #304: Added a new set API for TupleIdSequence.

Posted by jianqiao <gi...@git.apache.org>.

Github user jianqiao commented on the issue:

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


---

[GitHub] incubator-quickstep pull request #304: Added a new set API for TupleIdSequen...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---