This is an automated email from the ASF dual-hosted git repository. jenniferdai pushed a commit to branch errorMessage in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit 1898ac158643c5e8aace7a7c9ed12f411bc47f45 Author: Jennifer Dai <[email protected]> AuthorDate: Wed Jun 26 17:40:35 2019 -0700 Adding error if table does not exist to avoid null pointer --- .../src/main/java/org/apache/pinot/common/config/TableConfig.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java b/pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java index 2f4ee1c..2a9fe50 100644 --- a/pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java +++ b/pinot-common/src/main/java/org/apache/pinot/common/config/TableConfig.java @@ -112,6 +112,8 @@ public class TableConfig { public static TableConfig fromJsonConfig(JsonNode jsonConfig) throws IOException { + Preconditions + .checkState(jsonConfig != null, "Table does not exist"); // Mandatory fields JsonNode jsonTableType = jsonConfig.get(TABLE_TYPE_KEY); Preconditions --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
