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