Compile failed with tpch.tgz: invalid block type

2017-01-05 Thread WeiWan
branch: 1.8.0 I compile the drill source code with `mvn clean install -DskipTests`, and failed with errors below. After rerun with `-X` debug mode, I found this file (tpch.tgz) was downloaded from `http://apache-drill.s3.amazonaws.com/files/sf-0.01_tpc-h_parquet_typed.tgz`

[GitHub] drill issue #713: DRILL-3562: Query fails when using flatten on JSON data wh...

2017-01-05 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/713 Looks good. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, o

[jira] [Resolved] (DRILL-5176) Rest API for stats.json gives HTTP 404 Not Found error

2017-01-05 Thread Krystal (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5176?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Krystal resolved DRILL-5176. Resolution: Not A Problem Thanks Arina for the clarification. Marking jira as resolved. > Rest API for sta

[GitHub] drill pull request #713: DRILL-3562: Query fails when using flatten on JSON ...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/713#discussion_r94814315 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/flatten/FlattenRecordBatch.java --- @@ -305,12 +306,23 @@ protected boolean set

Re: Compile failed with tpch.tgz: invalid block type

2017-01-05 Thread Chun Chang
The FW might be the cause, but you can download manually so I would also look at your maven version. My version Apache Maven 3.1.1 seems works but I am outside of China. On Thu, Jan 5, 2017 at 1:35 AM, WeiWan wrote: > branch: 1.8.0 > > I compile the drill source code with `mvn clean install -Dsk

Re: Compile failed with tpch.tgz: invalid block type

2017-01-05 Thread Chunhui Shi
Or you want to delete tpch-sample-data from your local mvn cache (~/.m2/repository/org/apache/drill/contrib/data/tpch-sample-data) and try it again. The previous download could be corrupted or did not succeed. On Thu, Jan 5, 2017 at 11:02 AM, Chun Chang wrote: > The FW might be the cause, but yo

[GitHub] drill issue #715: DRILL-5105: remove buffer size checking in getBuffers sinc...

2017-01-05 Thread chunhui-shi
Github user chunhui-shi commented on the issue: https://github.com/apache/drill/pull/715 Keep original code as commented out per offline discussion with Paul. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your proje

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94857201 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,18 +245,85 @@ public SchemaPlus getDefaultSchema(SchemaP

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94858477 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JsonRecordWriter.java --- @@ -82,6 +84,9 @@ public void init(Map writerOption

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94856856 --- Diff: distribution/src/resources/drill-override-example.conf --- @@ -184,7 +184,12 @@ drill.exec: { # Set this property if custom absolute ro

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94858741 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetRecordWriter.java --- @@ -382,4 +391,20 @@ public void cleanup() throws

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94858983 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/TemporaryTablesAutomaticDropTest.java --- @@ -0,0 +1,99 @@ +/** --- End diff

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94859288 --- Diff: exec/rpc/src/main/java/org/apache/drill/exec/rpc/RpcBus.java --- @@ -164,6 +164,8 @@ public void operationComplete(ChannelFuture future) throws E

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94826816 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -54,18 +66,41 @@ private Map properties; privat

[GitHub] drill pull request #666: DRILL-4956: Temporary tables support

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/666#discussion_r94857634 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java --- @@ -207,18 +245,85 @@ public SchemaPlus getDefaultSchema(SchemaP

[GitHub] drill pull request #719: DRILL-5172: Display elapsed time for queries in the...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/719#discussion_r94860865 --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl --- @@ -106,7 +106,13 @@ STATE: ${model.getProfile().getState().name()} F

[GitHub] drill pull request #719: DRILL-5172: Display elapsed time for queries in the...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/719#discussion_r94860581 --- Diff: exec/java-exec/src/main/resources/rest/profile/list.ftl --- @@ -64,6 +65,8 @@ ${query.getState()}

[GitHub] drill pull request #719: DRILL-5172: Display elapsed time for queries in the...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/719#discussion_r94860776 --- Diff: exec/java-exec/src/main/resources/rest/profile/profile.ftl --- @@ -106,7 +106,13 @@ STATE: ${model.getProfile().getState().name()} F

[GitHub] drill issue #701: DRILL-4963: Sync remote and local function registries befo...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/701 General comment. Wasn't the design supposed to be that the lazy lookup would check for the function both in planning ("Calcite") and execution times? The whole point of the lazy init was to avoid

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94874974 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/ClassBuilder.java --- @@ -64,14 +67,12 @@ * local variables. Have fun! *

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94875162 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/ClassBuilder.java --- @@ -82,18 +83,18 @@ private final DrillConfig confi

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94875353 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/CodeCompiler.java --- @@ -117,18 +200,24 @@ public CodeCompiler(final DrillConfig conf

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94876735 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/CodeCompiler.java --- @@ -153,4 +242,17 @@ public GeneratedClassEntry(final Class claz

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94877069 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/QueryClassLoader.java --- @@ -58,6 +58,19 @@ public void injectByteCode(String classN

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94877010 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/compile/ClassCompilerSelector.java --- @@ -119,13 +119,18 @@ public ClassCompilerSelector(Cla

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94879149 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/ClassGenerator.java --- @@ -374,6 +388,124 @@ public HoldingContainer declare(MajorType t

[GitHub] drill pull request #716: DRILL-5116: Enable generated code debugging in each...

2017-01-05 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/716#discussion_r94879598 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -167,24 +167,61 @@ drill.exec: { debug: true, janino_maxsize: 262144,

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94696573 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/security/plain/PlainServer.java --- @@ -0,0 +1,175 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94879981 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94849762 --- Diff: protocol/src/main/protobuf/User.proto --- @@ -111,6 +115,21 @@ message BitToUserHandshake { optional string errorId = 4; optional string

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94867052 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestKerberosSaslAuthentication.java --- @@ -0,0 +1,239 @@ +/** + * Licen

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94701486 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserServer.java --- @@ -246,28 +163,78 @@ protected void handle(UserClientConnectionImpl

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94700422 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserAuthenticationUtil.java --- @@ -0,0 +1,255 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94880411 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.hpp --- @@ -522,6 +534,13 @@ class DrillClientImpl : public DrillClientImplBase{ exec:

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94881310 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94866946 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -17,7 +17,10 @@ */ package org.apache.drill.e

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94825267 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserClientAuthenticationHandler.java --- @@ -0,0 +1,229 @@ +/** + * Licensed to t

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94689336 --- Diff: common/src/main/java/org/apache/drill/common/KerberosUtil.java --- @@ -0,0 +1,62 @@ +/** + * Licensed to the Apache Software Foundation (ASF) u

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94880079 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94864855 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestCustomUserAuthenticator.java --- @@ -56,16 +68,27 @@ public void positiveUse

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94695872 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/security/plain/PlainServer.java --- @@ -0,0 +1,175 @@ +/** + * Licensed to the Apache

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94862522 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/security/TestSaslExchange.java --- @@ -0,0 +1,63 @@ +/** + * Licensed to the Apache So

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r94700671 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserAuthenticationUtil.java --- @@ -0,0 +1,255 @@ +/** + * Licensed to the Apache

Re: Compile failed with tpch.tgz: invalid block type

2017-01-05 Thread WeiWan
I use Maven 3.3.9. And I tried again this morning and found it worked. Maybe there is some problem with my network. Sorry for that. Regards Flow Wei > On Jan 6, 2017, at 03:16, Chunhui Shi wrote: > > Or you want to delete tpch-sample-data from your local mvn cache > (~/.m2/repository/org/ap