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.


---

Reply via email to