[GitHub] incubator-quickstep pull request #244: Fix a problem in CopyGroupList + mino...

2017-05-03 Thread zuyu
Github user zuyu commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/244#discussion_r114640556 --- Diff: storage/tests/SplitRowStoreTupleStorageSubBlock_unittest.cpp --- @@ -761,61 +761,89 @@ TEST_P(SplitRowStoreTupleStorageSubBlockTest,

[GitHub] incubator-quickstep pull request #244: Fix a problem in CopyGroupList + mino...

2017-05-03 Thread jianqiao
GitHub user jianqiao opened a pull request: https://github.com/apache/incubator-quickstep/pull/244 Fix a problem in CopyGroupList + minor style fixes This PR adjusts the implementation of `CopyGroupList::merge_contiguous()` to handle some cases correctly. E.g. let `R`be a