diff --git a/common/src/main/java/org/apache/drill/common/exceptions/UserExceptionUtils.java b/common/src/main/java/org/apache/drill/common/exceptions/UserExceptionUtils.java index 1d9fbff6f3b..15b6ec5f8a6 100644 --- a/common/src/main/java/org/apache/drill/common/exceptions/UserExceptionUtils.java +++ b/common/src/main/java/org/apache/drill/common/exceptions/UserExceptionUtils.java @@ -33,7 +33,8 @@ private static String decorateHint(final String text) { return String.format("[Hint: %s]", text); } public static String getUserHint(final Throwable ex) { - if (ex.getMessage().startsWith("Error getting user info for current user")) { + final String message = ex.getMessage(); + if (message != null && message.startsWith("Error getting user info for current user")) { //User does not exist hint return decorateHint(USER_DOES_NOT_EXIST); } else { diff --git a/contrib/storage-phoenix/pom.xml b/contrib/storage-phoenix/pom.xml index e21dd3a9b68..557edfbddbe 100644 --- a/contrib/storage-phoenix/pom.xml +++ b/contrib/storage-phoenix/pom.xml @@ -29,9 +29,9 @@ Drill : Contrib : Storage : Phoenix - 5.1.3 - - 2.4.17 + 5.2.1 + + 2.5.10 false @@ -120,11 +120,6 @@ - - org.apache.phoenix - phoenix-hbase-compat-2.4.1 - ${phoenix.version} - org.apache.phoenix phoenix-core @@ -198,7 +193,7 @@ org.apache.phoenix - phoenix-hbase-compat-2.4.0 + phoenix-hbase-compat-2.4.1 ${phoenix.version} test @@ -335,6 +330,12 @@ + + org.apiguardian + apiguardian-api + 1.1.2 + test + diff --git a/contrib/storage-phoenix/src/main/java/org/apache/drill/exec/store/phoenix/PhoenixStoragePlugin.java b/contrib/storage-phoenix/src/main/java/org/apache/drill/exec/store/phoenix/PhoenixStoragePlugin.java index daf4e480fa7..de0b8514759 100644 --- a/contrib/storage-phoenix/src/main/java/org/apache/drill/exec/store/phoenix/PhoenixStoragePlugin.java +++ b/contrib/storage-phoenix/src/main/java/org/apache/drill/exec/store/phoenix/PhoenixStoragePlugin.java @@ -43,12 +43,12 @@ import org.apache.drill.exec.store.phoenix.rules.PhoenixConvention; import com.fasterxml.jackson.core.type.TypeReference; -import org.apache.drill.exec.util.ImpersonationUtil; import com.google.common.cache.CacheBuilder; import com.google.common.cache.CacheLoader; import com.google.common.cache.LoadingCache; +import com.google.common.collect.ImmutableSet; +import org.apache.drill.exec.util.ImpersonationUtil; import org.apache.hadoop.security.UserGroupInformation; -import org.apache.tephra.shaded.com.google.common.collect.ImmutableSet; public class PhoenixStoragePlugin extends AbstractStoragePlugin {