924060929 commented on PR #68811: URL: https://github.com/apache/doris/pull/68811#issuecomment-6136742738
在最新 head `f98cd864eb26` 复查后，确认 [此前两处非阻塞的日志退化](https://github.com/apache/doris/pull/68811#issuecomment-6121809195) 已修复：普通 catalog 异常恢复了原因和堆栈；JDBC 连接池清理失败保留了异常类型和位置，同时避免输出驱动原始消息。新增测试也覆盖了这两项修复，本次增量没有发现新的阻塞问题。 剩下的是已有的两个最终输出点问题：[语句执行日志](https://github.com/apache/doris/pull/68811#discussion_r4246658720) 和 [流式任务日志](https://github.com/apache/doris/pull/68811#discussion_r4246659312)。当前 `StmtExecutor.java:959` 仍直接打印 `e`，`StreamingInsertJob.java:276` 仍直接打印 `ex`；顶层 message 脱敏后，日志仍会展开保留的原始 cause。因此驱动消息若包含配置中的密码或 JDBC URL，仍可能从 cause 再次输出。这里只依据源码调用链确认，未做运行时复现。 建议把通用诊断策略查找和完整异常渲染放在 FE 公共工具中，由 statement、job 和 catalog 的最终出口共用；JDBC connector 继续负责具体脱敏策略。OceanBase 校验也需要把策略保留到最终输出，而不只处理 `JobException` 的顶层消息。请补这两个真实日志出口的测试，检查完整异常链中的远端错误码保留、原始敏感值不出现。 这与作者说明一致：本次提交修复了两处非阻塞退化，尚未解决另外的 statement/job 输出问题。建议完成这部分后再合并。本轮为源码复审，未本地重跑构建、单测或回归。 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
