Aias00 opened a new pull request, #2963: URL: https://github.com/apache/rocketmq-dashboard/pull/2963
## Summary - Replace in-memory LLM config overrides with persisted General Settings reads so runtime state cannot diverge from stored state. - Preserve LLM-only fields (`llmEngine`, `maxTokens`, `temperature`) when saving General Settings from clients that omit them. - Persist `apiKey` even when explicitly cleared while keeping it WRITE_ONLY in API responses. - Add a `settings_key` singleton business key with a unique constraint, plus an upgrade script for existing MySQL volumes and concurrent first-write recovery. ## Replacement PRs Replaces #2418 and #2290. ## Issue References Refs #813 and #2289. ## Notes This keeps the API key persistence behavior already introduced on `rocketmq-studio`; it only removes the remaining runtime override path and folds in the settings singleton guard. Source PRs should stay open until this replacement PR is confirmed mergeable. ## Verification - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -Dtest=MybatisPlusSettingsRepositoryTest,LlmConfigServiceTest,SettingsServiceTest,SettingsControllerTest,DemoDataSqlCompatibilityTest test` - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -DskipTests package` - `git diff --check` -- 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]
