This introduces the changes to the sort operator and dependencies (tests and benchmarks) from https://github.com/bakoe/hyrise/pull/11/ replacing the previous sort operator implementation.
TODO
[x] remove second, backwards-compatible constructor of sort operator and for that, adjust all usages to use the new interface
[x] fix LQPTranslatorTest.Sort (somehow input_table_left() of the sort operator used in there is a nullptr leading to a crash when trying to fill _sort_definition_data_types)
[x] fix OperatorDeepCopyTest.DeepCopySort (also failing due to the reason above, also see inline comment)
[x] transfer benchmark changes from src/benchmark/operators/sort_new_benchmark.cpp and src/benchmark/operators/sort_benchmark.cpp
Description
This introduces the changes to the sort operator and dependencies (tests and benchmarks) from https://github.com/bakoe/hyrise/pull/11/ replacing the previous sort operator implementation.
TODO
LQPTranslatorTest.Sort(somehowinput_table_left()of the sort operator used in there is anullptrleading to a crash when trying to fill_sort_definition_data_types)OperatorDeepCopyTest.DeepCopySort(also failing due to the reason above, also see inline comment)src/benchmark/operators/sort_new_benchmark.cppandsrc/benchmark/operators/sort_benchmark.cpp