amrishlal edited a comment on pull request #6998:
URL: https://github.com/apache/incubator-pinot/pull/6998#issuecomment-867230470
> Please also update the PR description queries where the column is missing
in `json_match`
Done
> Please also update the PR description queries where the column is missing
in json_match
Done
> Please evaluate the overhead of this new optimizer to the existing queries
before merging
I have added the following check at the beginning of
JsonStatementOptimizer.optimize function:
```
// If schema doesn't have any JSON columns, there is no need to run this
optimizer.
if (schema == null || !schema.hasJSONColumn()) {
return;
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
