Repository: hive Updated Branches: refs/heads/master fb85336f7 -> 0d8323357
http://git-wip-us.apache.org/repos/asf/hive/blob/0d832335/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java ---------------------------------------------------------------------- diff --git a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java index dcc626c..75fbfa2 100644 --- a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java +++ b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java @@ -767,7 +767,7 @@ public interface RawStore extends Configurable { WMFullResourcePlan getActiveResourcePlan() throws MetaException; - boolean validateResourcePlan(String name) + List<String> validateResourcePlan(String name) throws NoSuchObjectException, InvalidObjectException, MetaException; void dropResourcePlan(String name) throws NoSuchObjectException, MetaException; http://git-wip-us.apache.org/repos/asf/hive/blob/0d832335/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java ---------------------------------------------------------------------- diff --git a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java index 1f6d900..da518ab 100644 --- a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java +++ b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java @@ -2412,7 +2412,7 @@ public class CachedStore implements RawStore, Configurable { } @Override - public boolean validateResourcePlan(String name) + public List<String> validateResourcePlan(String name) throws NoSuchObjectException, InvalidObjectException, MetaException { return rawStore.validateResourcePlan(name); } http://git-wip-us.apache.org/repos/asf/hive/blob/0d832335/standalone-metastore/src/main/thrift/hive_metastore.thrift ---------------------------------------------------------------------- diff --git a/standalone-metastore/src/main/thrift/hive_metastore.thrift b/standalone-metastore/src/main/thrift/hive_metastore.thrift index 3319b1e..1085ce5 100644 --- a/standalone-metastore/src/main/thrift/hive_metastore.thrift +++ b/standalone-metastore/src/main/thrift/hive_metastore.thrift @@ -1128,7 +1128,7 @@ struct WMValidateResourcePlanRequest { } struct WMValidateResourcePlanResponse { - 1: optional bool isValid; + 1: optional list<string> errors; } struct WMDropResourcePlanRequest { http://git-wip-us.apache.org/repos/asf/hive/blob/0d832335/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java ---------------------------------------------------------------------- diff --git a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java index 6121c9b..1e4fe5d 100644 --- a/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java +++ b/standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java @@ -982,9 +982,9 @@ public class DummyRawStoreForJdoConnection implements RawStore { } @Override - public boolean validateResourcePlan(String name) + public List<String> validateResourcePlan(String name) throws NoSuchObjectException, InvalidObjectException, MetaException { - return false; + return Collections.emptyList(); } @Override
