Github user zuyu commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/308#discussion_r143825045 --- Diff: storage/InsertDestinationInterface.hpp --- @@ -104,7 +104,7 @@ class InsertDestinationInterface { * insertion from ValueAccessor even when partially full. **/ virtual void bulkInsertTuples(ValueAccessor *accessor, - bool always_mark_full = false) = 0; + const bool always_mark_full = false) = 0; --- End diff -- It is required by the first pass of the multi-pass sort.
---