You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Vladimir Sitnikov (JIRA)" <ji...@apache.org> on 2018/08/10 21:39:00 UTC
[jira] [Created] (CALCITE-2463) Silence ERROR logs from
CalciteException, SqlValidatorException
Vladimir Sitnikov created CALCITE-2463:
------------------------------------------
Summary: Silence ERROR logs from CalciteException, SqlValidatorException
Key: CALCITE-2463
URL: https://issues.apache.org/jira/browse/CALCITE-2463
Project: Calcite
Issue Type: Bug
Components: core
Affects Versions: 1.17.0
Reporter: Vladimir Sitnikov
Assignee: Julian Hyde
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.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)