You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by je...@apache.org on 2019/06/27 00:40:49 UTC

[incubator-pinot] 01/01: Adding error if table does not exist to avoid null pointer

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 b132c280b1fa20564ae9e04420baa303a08631fe
Author: Jennifer Dai <jd...@linkedin.com>
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      | 3 +++
 1 file changed, 3 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..df60834 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,9 @@ public class TableConfig {
 
   public static TableConfig fromJsonConfig(JsonNode jsonConfig)
       throws IOException {
+    if (jsonConfig == null) {
+      throw new RuntimeException("Table does not exist");
+    }
     // Mandatory fields
     JsonNode jsonTableType = jsonConfig.get(TABLE_TYPE_KEY);
     Preconditions


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org