fix: wrong source type in pull query error message (#3885)

Fixes: #3523 Remove the 'table' in the error message 'Table `foo` is not materialized' as it won't always be a table: it could be a stream.
confluentinc · Nov 20, 2019 · 65523c7 · 65523c7
1 parent 475d039
commit 65523c7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...t/resources/rest-query-validation-tests/pull-queries-against-materialized-aggregates.json b/...t/resources/rest-query-validation-tests/pull-queries-against-materialized-aggregates.json
@@ -719,7 +719,7 @@
       ],
       "expectedError": {
         "type": "io.confluent.ksql.rest.entity.KsqlStatementErrorMessage",
-        "message": "Table 'X' is not materialized.",
+        "message": "'X' is not materialized.",
         "status": 400
       }
     },

diff --git a/ksql-rest-app/src/main/java/io/confluent/ksql/rest/server/execution/PullQueryExecutor.java b/ksql-rest-app/src/main/java/io/confluent/ksql/rest/server/execution/PullQueryExecutor.java
@@ -712,7 +712,7 @@ private static TableRowsEntity proxyTo(
   }
 
   private static KsqlException notMaterializedException(final SourceName sourceTable) {
-    return new KsqlException("Table '"
+    return new KsqlException("'"
         + sourceTable.toString(FormatOptions.noEscape()) + "' is not materialized. "
         + PullQueryValidator.NEW_QUERY_SYNTAX_SHORT_HELP
         + System.lineSeparator()