You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by pr...@apache.org on 2017/08/08 00:31:39 UTC

[1/3] drill git commit: DRILL-5675: Drill C++ Client reports incorrect datetime Literals Support Metadata

Repository: drill
Updated Branches:
  refs/heads/master 9d1d81573 -> 5c57b50f2


DRILL-5675: Drill C++ Client reports incorrect  datetime Literals Support Metadata

closes #878


Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/75b2136a
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/75b2136a
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/75b2136a

Branch: refs/heads/master
Commit: 75b2136ac8a7729c4660f8761162d8056afeba68
Parents: 9d1d815
Author: Rob Wu <ro...@gmail.com>
Authored: Mon Jul 17 17:12:39 2017 -0700
Committer: Paul Rogers <pr...@maprtech.com>
Committed: Mon Aug 7 14:41:19 2017 -0700

----------------------------------------------------------------------
 contrib/native/client/src/clientlib/metadata.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/drill/blob/75b2136a/contrib/native/client/src/clientlib/metadata.cpp
----------------------------------------------------------------------
diff --git a/contrib/native/client/src/clientlib/metadata.cpp b/contrib/native/client/src/clientlib/metadata.cpp
index ad08a40..843b307 100644
--- a/contrib/native/client/src/clientlib/metadata.cpp
+++ b/contrib/native/client/src/clientlib/metadata.cpp
@@ -837,11 +837,11 @@ static Drill::meta::DateTimeLiteralSupport dateTimeLiteralsSupport(const google:
 				break;
 
 			case exec::user::DL_INTERVAL_MINUTE:
-				result |= Drill::meta::DL_TIMESTAMP;
+				result |= Drill::meta::DL_INTERVAL_MINUTE;
 				break;
 
 			case exec::user::DL_INTERVAL_MINUTE_TO_SECOND:
-				result |= Drill::meta::DL_TIMESTAMP;
+				result |= Drill::meta::DL_INTERVAL_MINUTE_TO_SECOND;
 				break;
 
 			case exec::user::DL_INTERVAL_SECOND:


[2/3] drill git commit: DRILL-5686: Warning for sasl.max_wrapped_size contain incorrect syntax

Posted by pr...@apache.org.
DRILL-5686: Warning for sasl.max_wrapped_size contain incorrect syntax

closes #884


Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/5bb4b1f6
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/5bb4b1f6
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/5bb4b1f6

Branch: refs/heads/master
Commit: 5bb4b1f661edc7281245d2d2ceed8d339e75d702
Parents: 75b2136
Author: Sorabh Hamirwasia <sh...@maprtech.com>
Authored: Mon Jul 24 19:06:21 2017 -0700
Committer: Paul Rogers <pr...@maprtech.com>
Committed: Mon Aug 7 14:46:28 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/drill/exec/rpc/BitConnectionConfig.java     | 5 +++--
 .../org/apache/drill/exec/rpc/user/UserConnectionConfig.java    | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/drill/blob/5bb4b1f6/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BitConnectionConfig.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BitConnectionConfig.java b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BitConnectionConfig.java
index 7d9ebec..4f49993 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BitConnectionConfig.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/BitConnectionConfig.java
@@ -68,8 +68,9 @@ public abstract class BitConnectionConfig extends AbstractConnectionConfig {
             "bit.encryption.sasl.max_wrapped_size. Must be a positive integer in bytes with a recommended max value " +
             "of %s", RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE));
       } else if (maxWrappedSize > RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE) {
-        logger.warn("The configured value of bit.encryption.sasl.max_wrapped_size is too big. This may cause higher" +
-            " memory pressure. [Details: Recommended max value is %s]", RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE);
+        logger.warn("The configured value of bit.encryption.sasl.max_wrapped_size: {} is too big. This may cause " +
+            "higher memory pressure. [Details: Recommended max value is {}]",
+            maxWrappedSize, RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE);
       }
       encryptionContext.setMaxWrappedSize(maxWrappedSize);
 

http://git-wip-us.apache.org/repos/asf/drill/blob/5bb4b1f6/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserConnectionConfig.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserConnectionConfig.java b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserConnectionConfig.java
index 64ac6be..57ad4d5 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserConnectionConfig.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserConnectionConfig.java
@@ -61,8 +61,9 @@ class UserConnectionConfig extends AbstractConnectionConfig {
             "user.encryption.sasl.max_wrapped_size. Must be a positive integer in bytes with a recommended max value " +
             "of %s", RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE));
       } else if (maxWrappedSize > RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE) {
-        logger.warn("The configured value of user.encryption.sasl.max_wrapped_size is too big. This may cause higher" +
-            " memory pressure. [Details: Recommended max value is %s]", RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE);
+        logger.warn("The configured value of user.encryption.sasl.max_wrapped_size: {} is too big. This may cause " +
+            "higher memory pressure. [Details: Recommended max value is {}]",
+            maxWrappedSize, RpcConstants.MAX_RECOMMENDED_WRAPPED_SIZE);
       }
       encryptionContext.setMaxWrappedSize(maxWrappedSize);
 


[3/3] drill git commit: DRILL-5687: Disable TestMergeJoinWithSchemaChanges#testMissingAndNewColumns until DRILL-5612 is done

Posted by pr...@apache.org.
DRILL-5687: Disable TestMergeJoinWithSchemaChanges#testMissingAndNewColumns until DRILL-5612 is done

closes #885


Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/5c57b50f
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/5c57b50f
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/5c57b50f

Branch: refs/heads/master
Commit: 5c57b50f263fb0fda681fc566a11a41971fde14e
Parents: 5bb4b1f
Author: Arina Ielchiieva <ar...@gmail.com>
Authored: Tue Jul 25 14:37:31 2017 +0300
Committer: Paul Rogers <pr...@maprtech.com>
Committed: Mon Aug 7 14:47:41 2017 -0700

----------------------------------------------------------------------
 .../exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/drill/blob/5c57b50f/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java b/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java
index 08aae60..29aabe7 100644
--- a/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java
+++ b/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinWithSchemaChanges.java
@@ -1,4 +1,4 @@
-/**
+/*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
  * distributed with this work for additional information
@@ -201,8 +201,8 @@ public class TestMergeJoinWithSchemaChanges extends BaseTestQuery {
     builder.go();
   }
 
+  @Ignore("DRILL-5612")
   @Test
-  //@Ignore
   public void testMissingAndNewColumns() throws Exception {
     final File left_dir = new File(BaseTestQuery.getTempDir("mergejoin-schemachanges-left"));
     final File right_dir = new File(BaseTestQuery.getTempDir("mergejoin-schemachanges-right"));