[GitHub] Ben-Zvi closed pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi closed pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of prove

[GitHub] Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212790155 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project

[GitHub] kkhatua commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs

2018-08-24 Thread GitBox
kkhatua commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs URL: https://github.com/apache/drill/pull/1437#discussion_r212789862 ## File path: common/src/main/java/org/apache/drill/exec/metrics/CpuGaugeSet.java ## @@ -32,13 +

[GitHub] bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444#discussion_r212787951 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/Hash

[GitHub] bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444#discussion_r212788123 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/util/record/RecordBatchStats

[GitHub] bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444#discussion_r212788029 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/util/record/RecordBatchStats

[GitHub] bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444#discussion_r212699960 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/util/record/RecordBatchStats

[GitHub] bitblender commented on issue #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
bitblender commented on issue #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#issuecomment-415929261 @Ben-Zvi I am looking at the CI failures. T

[GitHub] bitblender commented on issue #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
bitblender commented on issue #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#issuecomment-415918229 @Ben-Zvi Thanks for your review. I have addressed your review comments.

[GitHub] Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212782192 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project

[GitHub] bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212781914 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/proj

[GitHub] bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212781917 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/proj

[GitHub] bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
bitblender commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212781844 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/proj

Re: [ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread salim achouche
Congrats Volodymyr! On Fri, Aug 24, 2018 at 11:32 AM Gautam Parai wrote: > Congratulations Vova! > > Gautam > > On Fri, Aug 24, 2018 at 10:59 AM, Khurram Faraaz wrote: > > > Congratulations Volodymyr! > > > > Regards, > > Khurram > > > > On Fri, Aug 24, 2018 at 10:25 AM, Hanumath Rao Maduri < >

Re: [Question] HiveStoragePlugin and NativeParquetRowGroupScan

2018-08-24 Thread Paul Rogers
Hi Tim, Can't recall the details on this. The phrase "the filesystem configuration" might be misleading. When executing, Drill must support multiple filesystems. I can have two different DFS configs, pointing to two different HDFS clusters (say) in a single query: SELECT ... FROM dfs1.`aFile.c

Re: [Question] HiveStoragePlugin and NativeParquetRowGroupScan

2018-08-24 Thread Timothy Farkas
Hi Paul / Vitalii Thanks for the info. I was asking about this because of https://issues.apache.org/jira/browse/DRILL-6609 in which some strange behavior was observed if the user defined fs.default.name in the HivePlugin config. I also saw that the filesystem specified in the HivePlugin config inf

[GitHub] Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212766463 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project

[GitHub] Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212762487 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project

[GitHub] Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified

2018-08-24 Thread GitBox
Ben-Zvi commented on a change in pull request #1442: DRILL-6688 Data batches for Project operator exceed the maximum specified URL: https://github.com/apache/drill/pull/1442#discussion_r212761189 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project

Re: [ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread Gautam Parai
Congratulations Vova! Gautam On Fri, Aug 24, 2018 at 10:59 AM, Khurram Faraaz wrote: > Congratulations Volodymyr! > > Regards, > Khurram > > On Fri, Aug 24, 2018 at 10:25 AM, Hanumath Rao Maduri > wrote: > > > Congratulations Volodymyr! > > > > Thanks, > > -Hanu > > > > On Fri, Aug 24, 2018 at

Re: [ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread Khurram Faraaz
Congratulations Volodymyr! Regards, Khurram On Fri, Aug 24, 2018 at 10:25 AM, Hanumath Rao Maduri wrote: > Congratulations Volodymyr! > > Thanks, > -Hanu > > On Fri, Aug 24, 2018 at 10:22 AM Paul Rogers > wrote: > > > Congratulations Volodymyr! > > Thanks, > > - Paul > > > > > > > > On Fri

Re: [ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread Hanumath Rao Maduri
Congratulations Volodymyr! Thanks, -Hanu On Fri, Aug 24, 2018 at 10:22 AM Paul Rogers wrote: > Congratulations Volodymyr! > Thanks, > - Paul > > > > On Friday, August 24, 2018, 5:53:25 AM PDT, Arina Ielchiieva < > ar...@apache.org> wrote: > > I am pleased to announce that Drill PMC invited

Re: [ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread Paul Rogers
Congratulations Volodymyr! Thanks, - Paul On Friday, August 24, 2018, 5:53:25 AM PDT, Arina Ielchiieva wrote: I am pleased to announce that Drill PMC invited Volodymyr Vysotskyi to the PMC and he has accepted the invitation. Congratulations Vova and thanks for your contributions! -

[GitHub] sachouche commented on issue #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
sachouche commented on issue #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444#issuecomment-415803914 @ilooner and @bitblender, can you please review this PR? Thanks!

[GitHub] sachouche opened a new pull request #1444: DRILL-6709: Extended the batch stats utility to other operators

2018-08-24 Thread GitBox
sachouche opened a new pull request #1444: DRILL-6709: Extended the batch stats utility to other operators URL: https://github.com/apache/drill/pull/1444 - Enhanced the Batch Stats utility to handle input / output operators as it was being used only by the Parquet reader - Modified all

[GitHub] arina-ielchiieva commented on issue #1443: DRILL-6705: Fix various failures in Crypto / Network / Phonetic functions when invalid input is given

2018-08-24 Thread GitBox
arina-ielchiieva commented on issue #1443: DRILL-6705: Fix various failures in Crypto / Network / Phonetic functions when invalid input is given URL: https://github.com/apache/drill/pull/1443#issuecomment-415760213 @cgivre could you please review? --

[GitHub] arina-ielchiieva opened a new pull request #1443: DRILL-6705: Fix various failures in Crypto / Network / Phonetic functions when invalid input is given

2018-08-24 Thread GitBox
arina-ielchiieva opened a new pull request #1443: DRILL-6705: Fix various failures in Crypto / Network / Phonetic functions when invalid input is given URL: https://github.com/apache/drill/pull/1443 1. aes_decrypt / aes_ecrypt - moved cyper init part into eval method since it not a constan

[ANNOUNCE] New PMC member: Volodymyr Vysotskyi

2018-08-24 Thread Arina Ielchiieva
I am pleased to announce that Drill PMC invited Volodymyr Vysotskyi to the PMC and he has accepted the invitation. Congratulations Vova and thanks for your contributions! - Arina (on behalf of Drill PMC)

[GitHub] arina-ielchiieva commented on issue #1397: DRILL-6633: Replace usage of Guava classes by JDK ones

2018-08-24 Thread GitBox
arina-ielchiieva commented on issue #1397: DRILL-6633: Replace usage of Guava classes by JDK ones URL: https://github.com/apache/drill/pull/1397#issuecomment-415748806 @vrozov / @vdiravka this PR has been open for a month already. I think you all agree that Volodymyr has done enormous work

[GitHub] arina-ielchiieva commented on a change in pull request #1425: DRILL-6647: Update Calcite version to 1.17.0

2018-08-24 Thread GitBox
arina-ielchiieva commented on a change in pull request #1425: DRILL-6647: Update Calcite version to 1.17.0 URL: https://github.com/apache/drill/pull/1425#discussion_r212613772 ## File path: pom.xml ## @@ -45,8 +45,8 @@ 18.0 2 1.10.0 -1.16.0-drill-r7 -

[GitHub] arina-ielchiieva commented on a change in pull request #1425: DRILL-6647: Update Calcite version to 1.17.0

2018-08-24 Thread GitBox
arina-ielchiieva commented on a change in pull request #1425: DRILL-6647: Update Calcite version to 1.17.0 URL: https://github.com/apache/drill/pull/1425#discussion_r212613772 ## File path: pom.xml ## @@ -45,8 +45,8 @@ 18.0 2 1.10.0 -1.16.0-drill-r7 -

[jira] [Created] (DRILL-6711) Publish Drill Calcite project artifacts to Apache maven repository

2018-08-24 Thread Arina Ielchiieva (JIRA)
Arina Ielchiieva created DRILL-6711: --- Summary: Publish Drill Calcite project artifacts to Apache maven repository Key: DRILL-6711 URL: https://issues.apache.org/jira/browse/DRILL-6711 Project: Apac

[GitHub] vrozov commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs

2018-08-24 Thread GitBox
vrozov commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs URL: https://github.com/apache/drill/pull/1437#discussion_r212612035 ## File path: common/src/main/java/org/apache/drill/exec/metrics/CpuGaugeSet.java ## @@ -32,13 +3

[GitHub] arina-ielchiieva commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs

2018-08-24 Thread GitBox
arina-ielchiieva commented on a change in pull request #1437: DRILL-6702: Disable CPU Reporting for non-HotSpot JDKs URL: https://github.com/apache/drill/pull/1437#discussion_r212609360 ## File path: common/src/main/java/org/apache/drill/exec/metrics/CpuGaugeSet.java ## @@