[jira] [Resolved] (DRILL-7303) Filter record batch does not handle zero-length batches

[ https://issues.apache.org/jira/browse/DRILL-7303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul Rogers resolved DRILL-7303. Resolution: Duplicate > Filter record batch does not handle zero-length batches >

[jira] [Resolved] (DRILL-7311) Partial fixes for empty batch bugs

[ https://issues.apache.org/jira/browse/DRILL-7311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul Rogers resolved DRILL-7311. Resolution: Duplicate > Partial fixes for empty batch bugs > -- >

[jira] [Resolved] (DRILL-7305) Multiple operators do not handle empty batches

[ https://issues.apache.org/jira/browse/DRILL-7305?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul Rogers resolved DRILL-7305. Resolution: Duplicate > Multiple operators do not handle empty batches >

[GitHub] [drill] paul-rogers opened a new pull request #1912: DRILL-7324: Final set of "batch count" fixes

paul-rogers opened a new pull request #1912: DRILL-7324: Final set of "batch count" fixes URL: https://github.com/apache/drill/pull/1912 Final set of fixes for batch count/record count issues. Enables vector checking for all operators (except statistics, which do not produce valid

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259556 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259595 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254883 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpGroupScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259685 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352253850 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352253816 ## File path: contrib/storage-http/README.md ## @@ -0,0 +1,31 @@ + +# Generic API Storage

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259705 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254617 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpGroupScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352253759 ## File path: contrib/storage-http/README.md ## @@ -0,0 +1,31 @@ + +# Generic API Storage

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254083 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352253268 ## File path: contrib/storage-http/README.md ## @@ -0,0 +1,31 @@ + +# Generic API Storage

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259793 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpSubScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254501 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpGroupScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259767 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259445 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpScanSpec.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259837 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpSubScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352260027 ## File path: contrib/storage-http/src/test/java/org/apache/drill/exec/store/http/TestHttpPlugin.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259264 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259654 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254315 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259476 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpScanSpec.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259891 ## File path: contrib/storage-http/src/main/resources/bootstrap-storage-plugins.json ## @@

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259260 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259746 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352254853 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpGroupScan.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API

paul-rogers commented on a change in pull request #1892: Drill-7437: Storage Plugin for Generic HTTP REST API URL: https://github.com/apache/drill/pull/1892#discussion_r352259504 ## File path: contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpGroupScan.java

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

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_r352252495 ## File path:

[GitHub] [drill] paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352251662 ## File path: common/src/test/java/org/apache/drill/test/BaseTest.java ##

[GitHub] [drill] paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352250720 ## File path: common/src/main/java/org/apache/drill/common/util/GuavaPatcher.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352250931 ## File path: common/src/main/java/org/apache/drill/common/util/GuavaPatcher.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352251049 ## File path: common/src/main/java/org/apache/drill/common/util/GuavaPatcher.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352251533 ## File path: common/src/test/java/org/apache/drill/test/BaseTest.java ##

[GitHub] [drill] paul-rogers commented on issue #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

paul-rogers commented on issue #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#issuecomment-559887767 The meta-question is why we still need this after all these years? Shouldn't we try to normalize our usage of

Re: [DISCUSS] 1.17.0 release

Hi all The release is coming, PR for DRILL-7450 created and approved, I'll merge it with other ready-to-commit PRs (including DRILL-6540 since DRILL-7393 is almost finished). Also, I'll try to

[GitHub] [drill] agozhiy commented on issue #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

agozhiy commented on issue #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#issuecomment-559850966 @vvysotskyi, thank you for review, I made the changes you requested.

[GitHub] [drill] arina-ielchiieva commented on issue #1904: DRILL-5844: Incorrect values of TABLE_TYPE returned from method DatabaseMetaData.getTables of JDBC API

arina-ielchiieva commented on issue #1904: DRILL-5844: Incorrect values of TABLE_TYPE returned from method DatabaseMetaData.getTables of JDBC API URL: https://github.com/apache/drill/pull/1904#issuecomment-559819512 @arjuntheprogrammer did you have a chance to fix test failures?

[GitHub] [drill] vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352104234 ## File path: exec/java-exec/src/test/java/org/apache/drill/BaseTestInheritance.java

[GitHub] [drill] vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352104680 ## File path: exec/java-exec/src/test/java/org/apache/drill/BaseTestInheritance.java

[GitHub] [drill] vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352110212 ## File path: exec/java-exec/src/test/java/org/apache/drill/BaseTestInheritance.java

[GitHub] [drill] vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b…

vvysotskyi commented on a change in pull request #1910: DRILL-7393: Revisit Drill tests to ensure that patching is executed b… URL: https://github.com/apache/drill/pull/1910#discussion_r352094027 ## File path:

[GitHub] [drill] vvysotskyi commented on issue #1911: DRILL-7208: Reuse root git.properties file

vvysotskyi commented on issue #1911: DRILL-7208: Reuse root git.properties file URL: https://github.com/apache/drill/pull/1911#issuecomment-559772432 @agozhiy, good question! Plugins declared for the same phase, will be executed in the order of their appearance in the pom file, but

[GitHub] [drill] agozhiy commented on issue #1911: DRILL-7208: Reuse root git.properties file

agozhiy commented on issue #1911: DRILL-7208: Reuse root git.properties file URL: https://github.com/apache/drill/pull/1911#issuecomment-559769539 @vvysotskyi, Could there be a race condition possible between maven-resources-plugin and git-commit-id-plugin?