[GitHub] drill issue #889: DRILL-5691: enhance scalar sub queries checking for the ca...

2017-10-28 Thread weijietong
Github user weijietong commented on the issue: https://github.com/apache/drill/pull/889 @amansinha100 I have removed the override of `getMaxRowCount` method of `TableScan` type to avoid your worry about unexpected results . This PR's target adjusts to just enhance the current

[GitHub] drill issue #1012: DRILL-5911: Upgrade esri-geometry-api version to 2.0.0 to...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1012 OK. LGTM. ---

[GitHub] drill issue #1012: DRILL-5911: Upgrade esri-geometry-api version to 2.0.0 to...

2017-10-28 Thread vvysotskyi
Github user vvysotskyi commented on the issue: https://github.com/apache/drill/pull/1012 @vrozov, sorry for the misunderstanding. Currently, it is also not supported. ---

Re: [ANNOUNCE] New Committer: Anil Kumar Batchu

2017-10-28 Thread Saurabh Mahapatra
Congratulations from all the users of Drill. Looking forward to your contributions. Best, Saurabh Sent from my iPhone > On Oct 28, 2017, at 10:58 AM, AnilKumar B wrote: > > Thank you all. > > Thanks & Regards, > B Anil Kumar. > >> On Fri, Oct 27, 2017 at 10:01 PM,

Re: [ANNOUNCE] New Committer: Anil Kumar Batchu

2017-10-28 Thread AnilKumar B
Thank you all. Thanks & Regards, B Anil Kumar. On Fri, Oct 27, 2017 at 10:01 PM, Jinfeng Ni wrote: > Congratulations and welcome, Anil! > > Jinfeng > > > On Thu, Oct 26, 2017 at 1:31 PM, Abhishek Girish > wrote: > > > Congratulations, Anil! > > > > On Thu,

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1013 @arina-ielchiieva I guess just logging an exception does not help unless it targets Java developers. Consider logging a message that has information regarding which authentication provider can't be

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1013 @vrozov as you can see from the changes, Volodymyr just has added additional information to the error message. I've seen from the previous discussions that you suggest just to log this

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1013 @vladimirtkach Who is the target of the message? If it is Java developers, the message makes sense. If it is for drill users or devops, the message is useless. ---

[GitHub] drill issue #1012: DRILL-5911: Upgrade esri-geometry-api version to 2.0.0 to...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1012 @vvysotskyi I refer to JSON format that ESRI supports by itself. ---

[GitHub] drill pull request #1016: DRILL-5913:solve the mixed processing of same func...

2017-10-28 Thread weijietong
GitHub user weijietong opened a pull request: https://github.com/apache/drill/pull/1016 DRILL-5913:solve the mixed processing of same functions with same inputRefs but di… `DrillReduceAggregatesRule` mix the processing of same functions with same inputRefs but different

[GitHub] drill pull request #936: DRILL-5772: Enable UTF-8 support in query string by...

2017-10-28 Thread arina-ielchiieva
Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/936#discussion_r147552950 --- Diff: exec/java-exec/src/test/resources/saffron.properties --- @@ -0,0 +1,23 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[jira] [Created] (DRILL-5913) DrillReduceAggregatesRule

2017-10-28 Thread weijie.tong (JIRA)
weijie.tong created DRILL-5913: -- Summary: DrillReduceAggregatesRule Key: DRILL-5913 URL: https://issues.apache.org/jira/browse/DRILL-5913 Project: Apache Drill Issue Type: Bug

[GitHub] drill issue #1012: DRILL-5911: Upgrade esri-geometry-api version to 2.0.0 to...

2017-10-28 Thread vvysotskyi
Github user vvysotskyi commented on the issue: https://github.com/apache/drill/pull/1012 @vrozov, If you meant [geojson](http://geojson.org) when saying about JSON format support, then my answer - not, it is not supported yet. ---

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vladimirtkach
Github user vladimirtkach commented on the issue: https://github.com/apache/drill/pull/1013 @sohami I'm getting this stack ``` java.lang.ExceptionInInitializerError at