FANNG1 commented on code in PR #8600:
URL: https://github.com/apache/gravitino/pull/8600#discussion_r2357594414


##########
docs/gravitino-mcp-server.md:
##########
@@ -53,7 +53,7 @@ Or start a HTTP MCP server by `uv run mcp_server --metalake 
test --uri http://12
 
 ### Docker instructions
 
-You could start Gravitino MCP server by docker image, `docker run -p 8000:8000 
apache/gravitino-mcp-server:latest --metalake test --transport http --mcp-url 
http://0.0.0.0:8000/mcp`. Please note that the MCP server in docker container 
doesn't support `stdio` transport mode.
+You could start Gravitino MCP server by docker image, `docker run -p 8000:8000 
--network=host apache/gravitino-mcp-server:latest --metalake test --transport 
http --mcp-url http://0.0.0.0:8000/mcp --uri=http://127.0.0.1:8090`. Please 
note that the MCP server in docker container doesn't support `stdio` transport 
mode.

Review Comment:
   Both Gravitino connectors and fileset client use `uri` to address Gravitino 
server,  how about using `gravitino-uri` to keep consistent?



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