Major Description
CalciteException and SqlValidatorException log exceptions at ERROR level.
This is not appropriate since
1) the exception might be expected by the caller
2) exception might contain sensitive information
3) ERROR level is not appropriate, since error in validation does not imply business-level error.
I suggest to remove `LOGGER.error` calls from Calcite code.
Logging exceptions at TRACE level is fine from my point of view, so client can enable trace logging if required for debug purposes.
Attachments
Issue Links
- links to