zhangshenghang commented on PR #12941: URL: https://github.com/apache/seatunnel/pull/12941#issuecomment-6330841116
Follow-up correction to my previous commit (a74dc4653): the LLM `api_key` text no longer lists `QIANFAN` — see bf80086f4. Reason: the option rule does declare a conditional for `QIANFAN`, but `LLMTransform.open()` sends that provider to the `default` branch and throws `Unsupported model provider`, so the rule is not backed by the runtime. This was established in the #12423 review (which follows the call chain into `LLMTransform`), and #12423 now removes QIANFAN from its LLM pages for the same reason. Keeping the requirement statement to `OPENAI`, `DOUBAO` and `MICROSOFT` avoids advertising a configuration that fails at startup. The Required cell is back to `yes` for consistency with the other pages. -- 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]
