Github user prashanth-vasudev commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/1228#discussion_r138710943
--- Diff: core/sql/executor/ex_sort.cpp ---
@@ -364,12 +365,15 @@ ExSortTcb::ExSortTcb(const ExSortTdb & sort_tdb,
sortType_.useIterQSForRunGeneration_ = 1;
break;
}
+
+ sortHeap_ = new(getHeap()) NAHeap("Sort Heap", (NAHeap *)getHeap(),
204800);
--- End diff --
Allocation of sortHeap_ here seems to be duplicate allocation since
sortHeap_ is already allocated at line 317. This line can be removed.
---
