Removed the virtual function call in InvokeOnAnyValueAccessor.
Project: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/commit/79710ca6 Tree: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/tree/79710ca6 Diff: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/diff/79710ca6 Branch: refs/heads/fix-iwyu Commit: 79710ca6c6b75410bf2c26b4646acbfc5d554d7c Parents: 696a783 Author: Zuyu Zhang <z...@cs.wisc.edu> Authored: Fri Oct 6 14:34:21 2017 -0500 Committer: Zuyu Zhang <z...@cs.wisc.edu> Committed: Mon Oct 9 11:37:16 2017 -0500 ---------------------------------------------------------------------- storage/SplitRowStoreTupleStorageSubBlock.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/79710ca6/storage/SplitRowStoreTupleStorageSubBlock.cpp ---------------------------------------------------------------------- diff --git a/storage/SplitRowStoreTupleStorageSubBlock.cpp b/storage/SplitRowStoreTupleStorageSubBlock.cpp index 5060208..9f5a839 100644 --- a/storage/SplitRowStoreTupleStorageSubBlock.cpp +++ b/storage/SplitRowStoreTupleStorageSubBlock.cpp @@ -343,7 +343,7 @@ tuple_id SplitRowStoreTupleStorageSubBlock::bulkInsertPartialTuplesImpl( InvokeOnAnyValueAccessor( accessor, - [&](auto *accessor) -> void { // NOLINT(build/c++11 + [&](auto *accessor) -> void { // NOLINT(build/c++11) BitVector<true> tuple_null_bitmap(tuple_slot, num_null_attrs_); const std::size_t nullmap_size = BitVector<true>::BytesNeeded(num_null_attrs_); @@ -410,7 +410,7 @@ tuple_id SplitRowStoreTupleStorageSubBlock::bulkInsertPartialTuplesImpl( max_num_tuples_to_insert += additional_tuples_insert; } } - } while (fill_to_capacity && !accessor->iterationFinishedVirtual() && + } while (fill_to_capacity && !accessor->iterationFinished() && num_tuples_inserted < max_num_tuples_to_insert); });