Github user asfgit closed the pull request at:
https://github.com/apache/drill/pull/932
---
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/932#discussion_r142017748
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/spill/RecordBatchSizer.java
---
@@ -74,53 +74,52 @@
public final int
Github user amansinha100 commented on a diff in the pull request:
https://github.com/apache/drill/pull/932#discussion_r141191139
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/spill/RecordBatchSizer.java
---
@@ -74,53 +74,52 @@
public final
Github user Ben-Zvi commented on a diff in the pull request:
https://github.com/apache/drill/pull/932#discussion_r137963963
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/managed/SortMemoryManager.java
---
@@ -680,9 +680,11 @@ public MergeTask
GitHub user paul-rogers opened a pull request:
https://github.com/apache/drill/pull/932
DRILL-5758: Fix for repeated columns; enable managed sort by default
Fix for DRILL-5443. Basically, the ârecord batch sizerâ did not handle
repeated columns correctly.
Enabled
