[calcite] branch master updated: [CALCITE-2762] Quidem env variable is always false if its name is separated by dot(.) (Hongze Zhang)

[vl...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

vladimirsitnikov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new b505ece  [CALCITE-2762] Quidem env variable is always false if its name is separated by dot(.) (Hongze Zhang)
b505ece is described below

commit b505ece7ff6cf906dfdc257848d85f381d103cb3
Author: hongzezhang <ho...@tencent.com>
AuthorDate: Thu Jan 3 18:23:16 2019 +0800

    [CALCITE-2762] Quidem env variable is always false if its name is separated by dot(.) (Hongze Zhang)
    
    fixes #987
---
 core/src/test/java/org/apache/calcite/test/QuidemTest.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/core/src/test/java/org/apache/calcite/test/QuidemTest.java b/core/src/test/java/org/apache/calcite/test/QuidemTest.java
index 12e5d2e..77d7191 100644
--- a/core/src/test/java/org/apache/calcite/test/QuidemTest.java
+++ b/core/src/test/java/org/apache/calcite/test/QuidemTest.java
@@ -55,6 +55,7 @@ import java.sql.DriverManager;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
+import java.util.function.Function;
 
 import static org.junit.Assert.fail;
 
@@ -77,8 +78,8 @@ public abstract class QuidemTest {
     case "jdk18":
       return System.getProperty("java.version").startsWith("1.8");
     case "fixed":
-      // Quidem requires a Guava function
-      return (com.google.common.base.Function<String, Object>) v -> {
+      // Quidem requires a Java 8 function
+      return (Function<String, Object>) v -> {
         switch (v) {
         case "calcite1045":
           return Bug.CALCITE_1045_FIXED;