[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198702179 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTableTemplate.java

[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198698608 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchMemoryManager.java

[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198712153 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggBatch.java

[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198711357 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggBatch.java

[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198697870 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggBatch.java

[GitHub] Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate

2018-06-27 Thread GitBox
Ben-Zvi commented on a change in pull request #1324: DRILL-6310: limit batch size for hash aggregate URL: https://github.com/apache/drill/pull/1324#discussion_r198711656 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/aggregate/HashAggBatch.java

[jira] [Created] (DRILL-6549) batch sizing for nested loop join

2018-06-27 Thread Padma Penumarthy (JIRA)
Padma Penumarthy created DRILL-6549: --- Summary: batch sizing for nested loop join Key: DRILL-6549 URL: https://issues.apache.org/jira/browse/DRILL-6549 Project: Apache Drill Issue Type:

[GitHub] cgivre commented on a change in pull request #1331: DRILL-6519: Add String Distance and Phonetic Functions

2018-06-27 Thread GitBox
cgivre commented on a change in pull request #1331: DRILL-6519: Add String Distance and Phonetic Functions URL: https://github.com/apache/drill/pull/1331#discussion_r198701570 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestStringDistanceFunctions.java

[GitHub] vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it

2018-06-27 Thread GitBox
vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it URL: https://github.com/apache/drill/pull/1264#discussion_r198691250 ## File path: drill-shaded/pom.xml ## @@ -0,0 +1,84 @@ + + +http://maven.apache.org/POM/4.0.0;

[GitHub] vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it

2018-06-27 Thread GitBox
vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it URL: https://github.com/apache/drill/pull/1264#discussion_r198690513 ## File path: drill-shaded/guava-shaded/pom.xml ## @@ -0,0 +1,147 @@ + +

[GitHub] vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it

2018-06-27 Thread GitBox
vrozov commented on a change in pull request #1264: DRILL-6422: Update guava to 23.0 and shade it URL: https://github.com/apache/drill/pull/1264#discussion_r198689121 ## File path: pom.xml ## @@ -3334,5 +3351,6 @@ exec drill-yarn distribution +

[GitHub] vdiravka commented on issue #1345: DRILL-6494: Drill Plugins Handler

2018-06-27 Thread GitBox
vdiravka commented on issue #1345: DRILL-6494: Drill Plugins Handler URL: https://github.com/apache/drill/pull/1345#issuecomment-400864708 @sohami Please review This is an automated message from the Apache Git Service. To

[GitHub] vdiravka opened a new pull request #1345: DRILL-6494: Drill Plugins Handler

2018-06-27 Thread GitBox
vdiravka opened a new pull request #1345: DRILL-6494: Drill Plugins Handler URL: https://github.com/apache/drill/pull/1345 - StoragePluginsHandler is added and implementation as StoragePluginsUpdater. It is used in the init() stage of StoragePluginRegistryImpl and updates storage

[jira] [Created] (DRILL-6548) IllegalStateException: Unexpected EMIT outcome received in buildSchema phase

2018-06-27 Thread Khurram Faraaz (JIRA)
Khurram Faraaz created DRILL-6548: - Summary: IllegalStateException: Unexpected EMIT outcome received in buildSchema phase Key: DRILL-6548 URL: https://issues.apache.org/jira/browse/DRILL-6548

[jira] [Resolved] (DRILL-6531) Errors in example for "Aggregate Function Interface" Boaz Ben-Zvi Fri 6/15, 5:54 PM Bridget Bevens

2018-06-27 Thread Bridget Bevens (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-6531?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bridget Bevens resolved DRILL-6531. --- Resolution: Fixed updated doc with the suggested changes. thanks, Bridget > Errors in

[jira] [Resolved] (DRILL-6393) Radians should take an argument (x)

2018-06-27 Thread Bridget Bevens (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-6393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bridget Bevens resolved DRILL-6393. --- Resolution: Fixed Made the changes and updates are published. Thanks, Bridget > Radians

[jira] [Created] (DRILL-6547) IllegalStateException: Tried to remove unmanaged buffer.

2018-06-27 Thread Robert Hou (JIRA)
Robert Hou created DRILL-6547: - Summary: IllegalStateException: Tried to remove unmanaged buffer. Key: DRILL-6547 URL: https://issues.apache.org/jira/browse/DRILL-6547 Project: Apache Drill

[GitHub] bitblender commented on issue #1335: DRILL-6529: Project Batch Sizing causes two LargeFileCompilation tests to timeout

2018-06-27 Thread GitBox
bitblender commented on issue #1335: DRILL-6529: Project Batch Sizing causes two LargeFileCompilation tests to timeout URL: https://github.com/apache/drill/pull/1335#issuecomment-400823880 @vvysotskyi @ilooner NUM_PROJECT_COLUMNS controls 3 other tests besides testTopN and

Re: [DISCUSSION] Travis build failures

2018-06-27 Thread Aman Sinha
Sounds good but why exclude the planning tests ? Only the TPC-H runtime tests should be excluded I think. On Wed, Jun 27, 2018 at 12:16 PM Timothy Farkas wrote: > +1 > > On Wed, Jun 27, 2018 at 10:00 AM, Vitalii Diravka < > vitalii.dira...@gmail.com > > wrote: > > > This is a topic from last

Re: [DISCUSSION] Travis build failures

2018-06-27 Thread Timothy Farkas
+1 On Wed, Jun 27, 2018 at 10:00 AM, Vitalii Diravka wrote: > This is a topic from last Hangout meeting. > > Sometimes Drill Travis Build fails because of job run expires. > Certainly, the right way is to accelerate Drill execution :) > > Nevertheless I believe we could consider excluding some

Re: [ANNOUNCE] New PMC member: Vitalii Diravka

2018-06-27 Thread Vitalii Diravka
Thank you all! It is a pleasure to work with all of you. I am sure Drill has a bright future. Therefore it's a honor for me to make contribution to this project. Kind regards Vitalii On Wed, Jun 27, 2018 at 9:38 AM Prasad Nagaraj Subramanya < prasadn...@gmail.com> wrote: > Congrats Vitali! >

Re: Tests failing

2018-06-27 Thread Vitalii Diravka
Hi Charles, Is your PR rebased onto the latest master? There was a commit (DRILL-6459), where UserBitShared$QueryProfile$Builder.setQueryId(Ljava/lang/String;) method is added. Did you have successful "mvn clean install -DskipTests"? Kind regards Vitalii On Wed, Jun 27, 2018 at 8:07 PM Charles

Tests failing

2018-06-27 Thread Charles Givre
Hello all, I just submitted a PR for some phonetic functions and I can’t get the unit tests to run on my local machine by themselves. When I submit the PR, the unit tests pass, but if I try to execute them with the following command. Any suggestions? mvn -Dtest=TestPhoneticFunctions test

[DISCUSSION] Travis build failures

2018-06-27 Thread Vitalii Diravka
This is a topic from last Hangout meeting. Sometimes Drill Travis Build fails because of job run expires. Certainly, the right way is to accelerate Drill execution :) Nevertheless I believe we could consider excluding some more tests from Travis Build. We can add all TPCH tests ( TestTpchLimit0,

[jira] [Created] (DRILL-6546) Allow unnest function with nested columns and complex expressions

2018-06-27 Thread Volodymyr Vysotskyi (JIRA)
Volodymyr Vysotskyi created DRILL-6546: -- Summary: Allow unnest function with nested columns and complex expressions Key: DRILL-6546 URL: https://issues.apache.org/jira/browse/DRILL-6546 Project:

[jira] [Created] (DRILL-6545) Projection Push down into Lateral Join operator.

2018-06-27 Thread Hanumath Rao Maduri (JIRA)
Hanumath Rao Maduri created DRILL-6545: -- Summary: Projection Push down into Lateral Join operator. Key: DRILL-6545 URL: https://issues.apache.org/jira/browse/DRILL-6545 Project: Apache Drill

[jira] [Created] (DRILL-6544) Timestamp value in Drill UI showed inconsistently with the same value retrieved from sqline

2018-06-27 Thread Anton Gozhiy (JIRA)
Anton Gozhiy created DRILL-6544: --- Summary: Timestamp value in Drill UI showed inconsistently with the same value retrieved from sqline Key: DRILL-6544 URL: https://issues.apache.org/jira/browse/DRILL-6544

Re: [ANNOUNCE] New PMC member: Vitalii Diravka

2018-06-27 Thread Prasad Nagaraj Subramanya
Congrats Vitali! On Tue, Jun 26, 2018 at 7:53 PM shi.chunhui wrote: > Congrats Vitalii! > -- > Sender:Parth Chandra > Sent at:2018 Jun 27 (Wed) 06:23 > To:dev > Subject:Re: [ANNOUNCE] New PMC member: Vitalii Diravka > >