alnzng opened a new pull request, #312:
URL: https://github.com/apache/flink-agents/pull/312

   <!--
   * Thank you very much for contributing to Flink Agents.
   * Please add the relevant components in the PR title. E.g., [api], 
[runtime], [java], [python], [hotfix], etc.
   -->
   
   <!-- Please link the PR to the relevant issue(s). Hotfix doesn't need this. 
-->
   Linked issue: #xxx
   
   ### Purpose of change
   
   <!-- What is the purpose of this change? -->
   
   We need to define `mcp_server` in Java `ResourceType` API, otherwise below 
error could occur if the app integrates with MCP server:
   
   ```
   py4j.protocol.Py4JJavaError: An error occurred while calling o125.invoke.
   : java.lang.IllegalArgumentException: Unknown ResourceType value: mcp_server
        at 
org.apache.flink.agents.api.resource.ResourceType.fromValue(ResourceType.java:58)
        at 
org.apache.flink.agents.plan.serializer.ResourceProviderJsonDeserializer.deserializePythonSerializableResourceProvider(ResourceProviderJsonDeserializer.java:104)
        at 
org.apache.flink.agents.plan.serializer.ResourceProviderJsonDeserializer.deserialize(ResourceProviderJsonDeserializer.java:66)
        at 
org.apache.flink.agents.plan.serializer.ResourceProviderJsonDeserializer.deserialize(ResourceProviderJsonDeserializer.java:45)
   ```
   
   
   ### Tests
   
   <!-- How is this change verified? -->
   Deploy a test app in local Flink cluster
   
   ### API
   
   <!-- Does this change touches any public APIs? -->
   - Add new `mcp_server` in `ResourceType.java`
   
   ### Documentation
   
   <!-- Should this change be covered by the user documentation?-->
   n/a
   


-- 
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]

Reply via email to