[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360518409 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/PriorityQueueCopierTe

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360426272 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360429440 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360525635 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortEmitOutcome.j

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360450237 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360511648 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360365959 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360443790 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360371811 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360394111 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360391704 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360515173 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360397128 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360515412 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSorter.java ###

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360526022 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSimpleExternalSor

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360443648 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360365118 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360423445 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360506932 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360368665 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360391462 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360464313 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360415588 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360365258 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360494754 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360364870 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360475113 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360404337 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360365036 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360485101 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360439104 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360507467 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360485448 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360511541 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360501525 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360518128 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/PriorityQueueCopierTe

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360516350 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSorter.java ###

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360472617 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360371678 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360427244 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360525569 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/xsort/TestSortImpl.java ##

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360517824 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/PriorityQueueCopier.j

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360508494 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360521056 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/PriorityQueueCopierTe

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r36065 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360425824 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.jav

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360508558 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360512752 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/MSortTemplate.java #

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360397396 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort

2019-12-20 Thread GitBox
ihuzenko commented on a change in pull request #1929: DRILL-6832: Remove the old "unmanaged" external sort URL: https://github.com/apache/drill/pull/1929#discussion_r360398804 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/BatchGroup.java

[GitHub] [drill] vvysotskyi opened a new pull request #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator

2019-12-21 Thread GitBox
vvysotskyi opened a new pull request #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator URL: https://github.com/apache/drill/pull/1938 - Removed excluding `com.fasterxml.woodstox:woodstox-core` since it is used in Hadoop classes, for example in

[GitHub] [drill] KazydubB commented on issue #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator

2019-12-22 Thread GitBox
KazydubB commented on issue #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator URL: https://github.com/apache/drill/pull/1938#issuecomment-568248470 LGTM, +1 This is an automated m

[GitHub] [drill] asfgit closed pull request #1936: DRILL-7490: LIMIT is not pushed to JDBC storage plugin

2019-12-22 Thread GitBox
asfgit closed pull request #1936: DRILL-7490: LIMIT is not pushed to JDBC storage plugin URL: https://github.com/apache/drill/pull/1936 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [drill] asfgit closed pull request #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator

2019-12-22 Thread GitBox
asfgit closed pull request #1938: DRILL-7494: Unable to connect to Drill using JDBC driver when using custom authenticator URL: https://github.com/apache/drill/pull/1938 This is an automated message from the Apache Git Servi

[GitHub] [drill] KazydubB commented on a change in pull request #1930: DRILL-7487: Removes the unused OUT_OF_MEMORY iterator status

2019-12-24 Thread GitBox
KazydubB commented on a change in pull request #1930: DRILL-7487: Removes the unused OUT_OF_MEMORY iterator status URL: https://github.com/apache/drill/pull/1930#discussion_r361159985 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatch.java

[GitHub] [drill] vvysotskyi opened a new pull request #1939: DRILL-7497: Fix warnings when starting Drill on Windows using Java 11

2019-12-24 Thread GitBox
vvysotskyi opened a new pull request #1939: DRILL-7497: Fix warnings when starting Drill on Windows using Java 11 URL: https://github.com/apache/drill/pull/1939 Jira - [DRILL-7497](https://issues.apache.org/jira/browse/DRILL-7497). `%DRILL_SHELL_JAVA_OPTS%` is expanded when if block

[GitHub] [drill] arina-ielchiieva commented on issue #1939: DRILL-7497: Fix warnings when starting Drill on Windows using Java 11

2019-12-24 Thread GitBox
arina-ielchiieva commented on issue #1939: DRILL-7497: Fix warnings when starting Drill on Windows using Java 11 URL: https://github.com/apache/drill/pull/1939#issuecomment-568778984 +1, LGTM This is an automated message from

[GitHub] [drill] ihuzenko opened a new pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-24 Thread GitBox
ihuzenko opened a new pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940 1. DRILL-7386 - added tests to TestHiveStructs. 2. DRILL-4527 - the DrillAvgVarianceConvertlet can't be removed without test failures. 3. DRILL-6215 - switched

[GitHub] [drill] cgivre opened a new pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-24 Thread GitBox
cgivre opened a new pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941 This PR addresses a few minor bugs in the Excel Format Plugin in which dates were not being parsed correctly. Also included in

[GitHub] [drill] arina-ielchiieva commented on issue #1937: DRILL-7230: Add README.md with instructions for release and release scripts

2019-12-25 Thread GitBox
arina-ielchiieva commented on issue #1937: DRILL-7230: Add README.md with instructions for release and release scripts URL: https://github.com/apache/drill/pull/1937#issuecomment-568910375 Please include info about Docker. Th

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323935 ## File path: contrib/storage-jdbc/src/test/java/org/apache/drill/exec/store/jdbc/TestJdbcWithMySQLandH2PluginsIT.

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323519 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361324076 ## File path: contrib/storage-jdbc/src/test/java/org/apache/drill/exec/store/jdbc/TestJdbcWithMySQLandH2PluginsIT.

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323751 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcStoragePlugin.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323795 ## File path: contrib/storage-jdbc/src/test/java/org/apache/drill/exec/store/jdbc/TestJdbcWithMySQLandH2PluginsIT.

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323267 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/CapitalizingJdbcSchema.java ###

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323595 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323645 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361324240 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/conversion/DrillTableKey.java ##

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323447 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323570 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323585 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361323411 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/DrillJdbcConvention.java ##

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361324545 ## File path: contrib/format-excel/src/test/resources/logback-test.xml ##

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361324464 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361324530 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel

[GitHub] [drill] arina-ielchiieva commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-25 Thread GitBox
arina-ielchiieva commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#issuecomment-568912112 @KazydubB I see you have pushed new commit, is PR ready for review? Please tag Paul if yes. Also please check if `or

[GitHub] [drill] vvysotskyi opened a new pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from

2019-12-25 Thread GitBox
vvysotskyi opened a new pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from developing branches URL: https://github.com/apache/drill/pull/1942 --

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left af

2019-12-25 Thread GitBox
arina-ielchiieva commented on a change in pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from developing branches URL: https://github.com/apache/drill/pull/1942#discussion_r361325725 #

[GitHub] [drill] vvysotskyi commented on a change in pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after ch

2019-12-25 Thread GitBox
vvysotskyi commented on a change in pull request #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from developing branches URL: https://github.com/apache/drill/pull/1942#discussion_r361326256 ###

[GitHub] [drill] vvysotskyi opened a new pull request #1943: Changes to use release API docs

2019-12-25 Thread GitBox
vvysotskyi opened a new pull request #1943: Changes to use release API docs URL: https://github.com/apache/drill/pull/1943 This PR contains two commits. The first one adds a link to JavaDocs into https://drill.apache.org/docs/ page and updates old links to 1.2.0 JDBC API. The second comm

[GitHub] [drill] cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361338312 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel/TestExcel

[GitHub] [drill] cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361338358 ## File path: contrib/format-excel/src/test/resources/logback-test.xml ## @@ -0,0 +

[GitHub] [drill] cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361338355 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel/TestExcel

[GitHub] [drill] cgivre commented on issue #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-25 Thread GitBox
cgivre commented on issue #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#issuecomment-568933123 @arina-ielchiieva I addressed all review comments and squashed commits. Thanks for the review and Happy Holidays!

[GitHub] [drill] arina-ielchiieva commented on issue #1943: Changes to use release API docs

2019-12-26 Thread GitBox
arina-ielchiieva commented on issue #1943: Changes to use release API docs URL: https://github.com/apache/drill/pull/1943#issuecomment-569054104 +1 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [drill] vvysotskyi merged pull request #1943: Changes to use release API docs

2019-12-26 Thread GitBox
vvysotskyi merged pull request #1943: Changes to use release API docs URL: https://github.com/apache/drill/pull/1943 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-26 Thread GitBox
arina-ielchiieva commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361451017 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel

[GitHub] [drill] arina-ielchiieva commented on issue #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-26 Thread GitBox
arina-ielchiieva commented on issue #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#issuecomment-569054996 +1, happy holidays to you, too :) This is a

[GitHub] [drill] cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column

2019-12-26 Thread GitBox
cgivre commented on a change in pull request #1941: DRILL-7495: Excel Reader Not Parsing Dates Correctly in First Column URL: https://github.com/apache/drill/pull/1941#discussion_r361452744 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel/TestExcel

[GitHub] [drill] ihuzenko commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0

2019-12-26 Thread GitBox
ihuzenko commented on a change in pull request #1940: DRILL-7406: Update Calcite to 1.21.0 URL: https://github.com/apache/drill/pull/1940#discussion_r361473590 ## File path: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcCatalogSchema.java ## @@ -

[GitHub] [drill] arina-ielchiieva commented on issue #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from d

2019-12-26 Thread GitBox
arina-ielchiieva commented on issue #1942: Update README.md with information on how to configure env for building site and update .gitignore to ignore files left after checkout from developing branches URL: https://github.com/apache/drill/pull/1942#issuecomment-569082379 +1 --

[GitHub] [drill] KazydubB commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
KazydubB commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#issuecomment-569130340 @paul-rogers, I've updated the PR, please have a look again. @arina-ielchiieva this is now fixed, thanks for pointing this ou

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361567680 ## File path: exec/vector/src/main/java/org/apache/drill/exec/vector/accessor/AbstractKeyAccessor.

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361565139 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/Container

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361551286 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/ColumnBui

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361566453 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/TupleStat

[GitHub] [drill] KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361600501 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/TupleState.j

[GitHub] [drill] KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361600935 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/ContainerSta

[GitHub] [drill] KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-12-26 Thread GitBox
KazydubB commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r361603427 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/resultSet/impl/ColumnBuilde

  1   2   3   4   5   6   7   8   9   10   >