kayx23 commented on code in PR #13187: URL: https://github.com/apache/apisix/pull/13187#discussion_r3056814604
########## docs/en/latest/plugins/ai-proxy-multi.md: ########## @@ -37,7 +37,7 @@ description: The ai-proxy-multi Plugin extends the capabilities of ai-proxy with The `ai-proxy-multi` Plugin simplifies access to LLM and embedding models by transforming Plugin configurations into the designated request format for OpenAI, DeepSeek, Azure, AIMLAPI, Anthropic, OpenRouter, Gemini, Vertex AI, and other OpenAI-compatible APIs. It extends the capabilities of [`ai-proxy`](./ai-proxy.md) with load balancing, retries, fallbacks, and health checks. -In addition, the Plugin also supports logging LLM request information in the access log, such as token usage, model, time to the first response, and more. +In addition, the Plugin also supports appending LLM request information to structured access log entries, such as token usage, model, time to the first response, and more. These structured entries are consumed by logging plugins such as `http-logger` and `kafka-logger`, and are separate from the debug messages written to `error.log`. Review Comment: The access log here meant: <img width="1048" height="489" alt="Image" src="https://github.com/user-attachments/assets/40d9161b-5ac0-4406-93fb-3f451032ec37"; /> I think this update has tweaked the original meaning a bit. The sentence could be updated to cover both the access logs and usage with loggers. WDYT? -- 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]
