wangzhigang1999 commented on PR #7530: URL: https://github.com/apache/kyuubi/pull/7530#issuecomment-4840614008
**Design note — `statementId`** `StatementInterceptContext#statementId()` is the operation-handle UUID, **pre-allocated before interception** and reused for the operation (instead of being self-generated afterwards). So one stable id spans intercept time → operation → result set → the handle the client gets. That's the correlation/join key for end-to-end audit and lineage, and for the planned result-set transform hook (second half of #7458). It's threaded down via a defaulted ctor param through `AbstractOperation`/`KyuubiOperation`/`ExecuteStatement`/`ExecutedCommandExec`, so no engine ripple. Also manually verified on a Kubernetes deployment: REWRITE/REJECT over Beeline (reject surfaces SQLState `42000`), `statementId` matches the operation handle, and neither batch nor a session-level override bypasses the `serverOnly` interceptor. -- 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]
