[GitHub] drill pull request #717: DRILL-5080: Memory-managed version of external sort

2017-01-01 Thread paul-rogers
GitHub user paul-rogers opened a pull request: https://github.com/apache/drill/pull/717 DRILL-5080: Memory-managed version of external sort Please see the DRILL-5080 JIRA, and subtasks, for reasons for revision, design spec and list of changes. Basically the idea is to beef up the l

Sort performance improvement

2017-01-01 Thread Paul Rogers
Hi All, Happy New Year! Recent experiments show that we can more than double in-memory sort performance by generating tighter code in the “inner loops” of the sort and merge implementations. The improved sort performance causes a 50% reduction in overall runtime (doubling of performance) for t

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-01 Thread paul-rogers
GitHub user paul-rogers opened a pull request: https://github.com/apache/drill/pull/716 DRILL-5116: Enable generated code debugging in each Drill operator DRILL-5052 added the ability to debug generated code. The reviewer suggested permitting the technique to be used for all Drill o

[GitHub] drill pull request #715: DRILL-5105: remove buffer size checking in getBuffe...

2017-01-01 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/715#discussion_r94288748 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java --- @@ -413,14 +413,9 @@ public RepeatedMapAccessor getAcc

[GitHub] drill pull request #715: DRILL-5105: remove buffer size checking in getBuffe...

2017-01-01 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/715#discussion_r94288743 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/vector/complex/MapVector.java --- @@ -134,10 +134,6 @@ public int getBufferSizeFor(final int val

[GitHub] drill pull request #715: DRILL-5105: remove buffer size checking in getBuffe...

2017-01-01 Thread chunhui-shi
GitHub user chunhui-shi opened a pull request: https://github.com/apache/drill/pull/715 DRILL-5105: remove buffer size checking in getBuffers since it is onl… …y for debug and the redundancy increases exponentially. You can merge this pull request into a Git repository by runnin