fix: treat wildcard Elasticsearch select as simple asterisk select: include * as part of RE to support elasticsearch indexes (#760) Thanks to @gauravlanjekar !

gauravlanjekar · glanjekar · web-flow · commit 099a8889e175 · 2020-06-27T21:34:38.000-07:00
Co-authored-by: gaurav lanjekar &lt;gaurav@leanspace.io&gt;
diff --git a/packages/cubejs-schema-compiler/adapter/BaseQuery.js b/packages/cubejs-schema-compiler/adapter/BaseQuery.js
@@ -957,7 +957,7 @@ class BaseQuery {
       return this.preAggregations.originalSqlPreAggregationTable(foundPreAggregation);
     }
     const evaluatedSql = this.evaluateSql(cube, this.cubeEvaluator.cubeFromPath(cube).sql);
-    const selectAsterisk = evaluatedSql.match(/^\s*select\s+\*\s+from\s+([a-zA-Z0-9_\-`".]+)\s*$/i);
+    const selectAsterisk = evaluatedSql.match(/^\s*select\s+\*\s+from\s+([a-zA-Z0-9_\-`".*]+)\s*$/i);
     if (selectAsterisk) {
       return selectAsterisk[1];
     }

Original file line number	Diff line number	Diff line change
`@@ -957,7 +957,7 @@ class BaseQuery {`
`957`	`957`	`return this.preAggregations.originalSqlPreAggregationTable(foundPreAggregation);`
`958`	`958`	`}`
`959`	`959`	`const evaluatedSql = this.evaluateSql(cube, this.cubeEvaluator.cubeFromPath(cube).sql);`
`960`		- const selectAsterisk = evaluatedSql.match(/^\sselect\s+\\s+from\s+([a-zA-Z0-9_\-`".]+)\s*$/i);
	`960`	+ const selectAsterisk = evaluatedSql.match(/^\sselect\s+\\s+from\s+([a-zA-Z0-9_\-`".]+)\s$/i);
`961`	`961`	`if (selectAsterisk) {`
`962`	`962`	`return selectAsterisk[1];`
`963`	`963`	`}`