[GitHub] drill pull request #996: DRILL-5878: TableNotFound exception is being report...

Github user HanumathRao commented on a diff in the pull request: https://github.com/apache/drill/pull/996#discussion_r147896927 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/store/dfs/TestFileSelection.java --- @@ -63,4 +63,17 @@ public void

[jira] [Created] (DRILL-5915) Streaming aggregate with limit query does not return

Boaz Ben-Zvi created DRILL-5915: --- Summary: Streaming aggregate with limit query does not return Key: DRILL-5915 URL: https://issues.apache.org/jira/browse/DRILL-5915 Project: Apache Drill

Re: Drill SASL Forward Compatibility

Regarding DRILL-5582, I see that fix as a breakage of the work to maintain compatibility for an newer client to connect to a older version of the server. Or put it differently: current (master) client does not connect anymore to a server not supporting SASL (<=1.9). Note that the client could

Drill SASL Forward Compatibility

Hi All, We recently added a check (as part of DRILL-5582) on DrillClient side to enforce that if client showed intent for authentication and Drillbit say's it doesn't require authentication then connection will fail with proper error message.

[jira] [Created] (DRILL-5914) CSV (text) reader fails to parse quoted newlines in trailing fields

Paul Rogers created DRILL-5914: -- Summary: CSV (text) reader fails to parse quoted newlines in trailing fields Key: DRILL-5914 URL: https://issues.apache.org/jira/browse/DRILL-5914 Project: Apache Drill

[GitHub] drill issue #976: DRILL-5797: Choose parquet reader from read columns

Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/976 @dprofeta, tried to commit this PR, but ran into multiple functional test failures: ``` Execution Failures:

[GitHub] drill pull request #978: DRILL-5842: Refactor and simplify the fragment, ope...

Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/978#discussion_r147842797 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/physical/unit/MiniPlanUnitTestBase.java --- @@ -360,7 +366,7 @@ public T

[GitHub] drill issue #978: DRILL-5842: Refactor and simplify the fragment, operator c...

Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/978 @jinfengni, sorry my description was a bit ambiguous. This PR has two goals: 1. Clean up the contexts based on what has been learned recently. 2. Evolve the contexts to allow

Re: Implicit columns and maps

Hi Jinfeng, Thanks much for your thoughts. To follow up on two topics. First, how would you suggest we handle the following: SELECT user, filename, `filename` FROM … Here, the project list handed to the scan operator will have three columns, two with name “filename”. I suppose we’d need to

[GitHub] drill pull request #1009: DRILL-5905: Exclude jdk-tools from project depende...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1009 ---

[GitHub] drill pull request #1013: DRILL-5910: Logging exception when custom Authenti...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1013 ---

[GitHub] drill pull request #1006: DRILL-5895: Add logging mongod exception when fail...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1006 ---

[GitHub] drill pull request #1012: DRILL-5911: Upgrade esri-geometry-api version to 2...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1012 ---

[GitHub] drill pull request #1010: DRILL-5906: java.lang.NullPointerException while q...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1010 ---

Re: [HANGOUT] Topics for 10/31/2017

I'll speak about unit testing: - Common mistakes that were made in unit tests - The soon to be merged temp directory test watcher classes - Making our Travis build run smoke tests with code coverage - Other misc improvements to be made Thanks, Tim From:

[GitHub] drill issue #996: DRILL-5878: TableNotFound exception is being reported for ...

Github user HanumathRao commented on the issue: https://github.com/apache/drill/pull/996 @arina-ielchiieva Thank you for the review comments. I have modified the code accordingly. Please let me know if anything needs to be changed. ---

[GitHub] drill pull request #996: DRILL-5878: TableNotFound exception is being report...

Github user HanumathRao commented on a diff in the pull request: https://github.com/apache/drill/pull/996#discussion_r147818548 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/SchemaUtilites.java --- @@ -77,6 +77,22 @@ public static SchemaPlus

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

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/936 ---

[GitHub] drill pull request #998: DRILL-5887: Display process user/groups info in Dri...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/998 ---

[GitHub] drill pull request #993: DRILL-5874: NPE in AnonWebUserConnection.cleanupSes...

Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/993 ---

[jira] [Resolved] (DRILL-1499) Different column order could appear in the result set for a schema-less select * query, even there are no changing schemas.

[ https://issues.apache.org/jira/browse/DRILL-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vitalii Diravka resolved DRILL-1499. Resolution: Resolved Assignee: Vitalii Diravka (was: Steven Phillips) Fix

Re: [HANGOUT] Topics for 10/31/2017

I'd suggest discussing migration to gitbox https://gitbox.apache.org/. For more details please see https://git.apache.org/. Thank you, Vlad On 10/30/17 09:26, Gautam Parai wrote: Hi, We will have a Drill hangout tomorrow (Tuesday Oct 31) at 10 AM Pacific Time. Please suggest topics by

Re: [GitHub] drill issue #971: Drill-5834 Add Networking Functions

I *think* I just did this correctly, but let me know if I didn’t. —C > On Oct 30, 2017, at 12:54, paul-rogers wrote: > > Github user paul-rogers commented on the issue: > >https://github.com/apache/drill/pull/971 > >Please squash commits. > > > ---

[GitHub] drill issue #1005: DRILL-5896: Handle HBase columns vector creation in the H...

Github user prasadns14 commented on the issue: https://github.com/apache/drill/pull/1005 @paul-rogers please review the changes ---

RE: [HANGOUT] Topics for 10/31/2017

I'd like a talk on 'how to get started debugging Drill.' Without a developer's guide or explicit instructions for getting a debug environment up and running, it's hard for a newcomer to get involved. -Original Message- From: Gautam Parai [mailto:gpa...@mapr.com] Sent: Monday, October

[GitHub] drill issue #971: Drill-5834 Add Networking Functions

Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/971 Please squash commits. ---

[HANGOUT] Topics for 10/31/2017

Hi, We will have a Drill hangout tomorrow (Tuesday Oct 31) at 10 AM Pacific Time. Please suggest topics by replying to this thread or bring them up during the hangout. Hangout link: https://plus.google.com/hangouts/_/event/ci4rdiju8bv04a64efj5fedd0lc Thanks, Gautam

[GitHub] drill issue #1013: DRILL-5910: Logging exception when custom AuthenticatorFa...

Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1013 LGTM ---

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134712 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134172 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583562 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/query/Query.java --- @@ -0,0 +1,182 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582647 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147585085 --- Diff: contrib/storage-opentsdb/src/test/java/org/apache/drill/store/openTSDB/TestDataHolder.java --- @@ -0,0 +1,177 @@ +/* + * Licensed

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147584976 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/schema/OpenTSDBSchemaFactory.java --- @@ -0,0 +1,126 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147584562 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/schema/OpenTSDBSchemaFactory.java --- @@ -0,0 +1,126 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583241 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/Util.java --- @@ -0,0 +1,55 @@ +/* + * Licensed to

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135322 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBGroupScan.java --- @@ -0,0 +1,220 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582484 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/OpenTSDBTypes.java --- @@ -0,0 +1,28 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147584411 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/schema/OpenTSDBSchemaFactory.java --- @@ -0,0 +1,126 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583993 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147584453 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/schema/OpenTSDBSchemaFactory.java --- @@ -0,0 +1,126 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134773 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583640 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583342 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/query/DBQuery.java --- @@ -0,0 +1,108 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582817 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBStoragePlugin.java --- @@ -0,0 +1,90 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133035 --- Diff: contrib/storage-opentsdb/src/test/resources/logback.xml --- @@ -0,0 +1,64 @@ + --- End diff -- Now in Drill we have common

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134878 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/schema/OpenTSDBSchemaFactory.java --- @@ -0,0 +1,126 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583417 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/query/DBQuery.java --- @@ -0,0 +1,108 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133289 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/OpenTSDB.java --- @@ -0,0 +1,61 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134351 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582692 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135774 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBScanSpec.java --- @@ -0,0 +1,35 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135479 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/DrillOpenTSDBTable.java --- @@ -0,0 +1,69 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147584018 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135112 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBGroupScan.java --- @@ -0,0 +1,220 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582988 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBSubScan.java --- @@ -0,0 +1,134 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133068 --- Diff: distribution/src/assemble/bin.xml --- @@ -99,7 +99,9 @@ org.apache.drill.contrib:drill-format-mapr

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133090 --- Diff: exec/java-exec/src/test/resources/drill-module.conf --- @@ -10,7 +10,7 @@ drill: { packages += "org.apache.drill.exec.testing",

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135440 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBGroupScan.java --- @@ -0,0 +1,220 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146135302 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBRecordReader.java --- @@ -0,0 +1,263 @@ +/* +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134207 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147582378 --- Diff: contrib/storage-opentsdb/README.md --- @@ -0,0 +1 @@ +# drill-storage-openTSDB --- End diff -- It would be nice if you add

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133808 --- Diff: contrib/storage-opentsdb/src/main/resources/bootstrap-storage-plugins.json --- @@ -0,0 +1,9 @@ +{ + "storage": { +

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146134226 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/client/services/ServiceImpl.java --- @@ -0,0 +1,176 @@

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r147583175 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/OpenTSDBSubScan.java --- @@ -0,0 +1,134 @@ +/* + *

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133164 --- Diff: contrib/storage-opentsdb/src/main/java/org/apache/drill/exec/store/openTSDB/Constants.java --- @@ -0,0 +1,31 @@ +/* + * Licensed

[GitHub] drill pull request #774: DRILL-5337: OpenTSDB storage plugin

Github user arina-ielchiieva commented on a diff in the pull request: https://github.com/apache/drill/pull/774#discussion_r146133055 --- Diff: distribution/pom.xml --- @@ -190,11 +190,21 @@ org.apache.drill.contrib +

[GitHub] drill issue #1013: DRILL-5910: Logging exception when custom AuthenticatorFa...

Github user vladimirtkach commented on the issue: https://github.com/apache/drill/pull/1013 @vrozov made the changes, please review. ---

[GitHub] drill pull request #1013: DRILL-5910: Add factory name to ClassNotFoundExcep...

Github user vladimirtkach commented on a diff in the pull request: https://github.com/apache/drill/pull/1013#discussion_r147655905 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/security/ClientAuthenticatorProvider.java --- @@ -57,17 +57,17 @@ private