GitHub user xintongsong added a comment to the discussion: [Feature] Dynamic Runtime Loading and Refresh of MCP Tools and Prompts
Thanks for preparing this, @yanand0909. There seems to be an overlap of efforts with @weiqingy #549. You guys may exchange ideas and figure out how to collaborate on this. Concerning the design doc, I had a brief look at the Proposed Solution and Public Interface parts, but not the Implementation Details. A few comments so far: 1. Why do we need the option `failFastOnStartup` for a MCP Server? In which case would the user want to not fail the job if MCP Server is unreachable? 2. `MCPServer` is not a public API. Users should neither implement it, nor directly call any methods on it. They are defined via the `@MCPServer` annotation and resource descriptors. GitHub link: https://github.com/apache/flink-agents/discussions/543#discussioncomment-15909753 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
