nevzheng commented on code in PR #12531:
URL: https://github.com/apache/gravitino/pull/12531#discussion_r3877204643


##########
docs/gravitino-mcp-server.md:
##########
@@ -134,19 +142,24 @@ Gravitino MCP server supports the following tools, and 
you could export tool by
 
 You could config Gravitino MCP server by arguments, `uv run mcp_server -h` 
shows the detailed information.
 
-| Argument          | Description                                              
                        | Default value               | Required |
-|-------------------|----------------------------------------------------------------------------------|-----------------------------|----------|
-| `--metalake`      | The Gravitino metalake name.                             
                        | none                        | Yes      |
-| `--gravitino-uri` | The URI of Gravitino server.                             
                        | `http://127.0.0.1:8090`     | No       |
-| `--transport`     | Transport protocol: stdio (local), http / 
streamable-http (Streamable HTTP).     | `stdio`                     | No       
|
-| `--mcp-url`       | The URL of MCP server if using HTTP transport.           
                        | `http://127.0.0.1:8000/mcp` | No       |
-| `--token`         | Static credential for Gravitino; or set 
`GRAVITINO_TOKEN`. See Authentication.   | none (anonymous)            | No     
  |
-| `--tls-cert`      | PEM certificate to serve the endpoint over HTTPS. 
Requires `--tls-key`.          | none                        | No       |
-| `--tls-key`       | PEM private key to serve the endpoint over HTTPS. 
Requires `--tls-cert`.         | none                        | No       |
+| Argument                   | Description                                     
                                                                 | Default 
value               | Required |
+|----------------------------|------------------------------------------------------------------------------------------------------------------|-----------------------------|----------|
+| `--metalake`               | The Gravitino metalake name.                    
                                                                 | none         
               | Yes      |
+| `--gravitino-uri`          | The URI of Gravitino server.                    
                                                                 | 
`http://127.0.0.1:8090`     | No       |
+| `--transport`              | Transport protocol: stdio (local), http / 
streamable-http (Streamable HTTP).                                    | `stdio` 
                    | No       |

Review Comment:
   Agree. Done — padded every Configuration column to the widest cell so 
`--no-service-identity-fallback` no longer breaks the pipes. Commit 76380827c.
   
   Nevin
   Sent from my 🤖 (Cursor)



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