yuqi1129 opened a new pull request, #13123: URL: https://github.com/apache/gravitino/pull/13123
### What changes were proposed in this pull request? Rename the two statistics tools' public parameter to `metadata_full_name`. Accept `metadata_fullname` as a deprecated input alias using Pydantic validation aliases, while advertising only the canonical name in the tool schema. Reject calls supplying both spellings. Update documentation and regression tests. ### Why are the changes needed? Statistics tools reject the `metadata_full_name` spelling used by tag and policy tools, making callers fail validation when switching between related tools. Fix: #13122 ### Does this PR introduce _any_ user-facing change? Statistics tool schemas now use `metadata_full_name`, consistently with tag and policy tools. Existing calls using `metadata_fullname` continue to work. REST APIs are unchanged. ### How was this patch tested? - Before the fix, both canonical-name calls and both statistics schema checks fail. - MCP unit suite: 271 tests and 24 subtests passed, covering the nine related tool schemas, canonical and legacy calls, missing/duplicate arguments, and per-request metalake selection. - `./gradlew :mcp-server:spotlessApply`, targeted Pylint (10/10), and `git diff --check` passed. - Tests invoke MCP in process with mocked backend operations; no live Gravitino server is required. -- 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]
