membphis commented on PR #13632: URL: https://github.com/apache/apisix/pull/13632#issuecomment-4850618213
One compatibility concern: this PR changes the existing exact-cache hit header from `X-AI-Cache-Status: HIT` to `X-AI-Cache-Status: HIT-L1`, even when the semantic layer is not enabled. The `-L1` suffix also feels a bit odd as a user-facing status, and this may break existing clients, tests, dashboards, or log parsing that already check for `HIT`. Could we keep the exact-cache status as `HIT` for backward compatibility, and only use a distinct value such as `HIT-L2` when the semantic layer serves the response? Another option would be to add a separate cache-layer header if we need to expose the serving layer explicitly. -- 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]
