FreeOnePlus opened a new pull request, #156: URL: https://github.com/apache/doris-mcp-server/pull/156
## Summary - use `SHOW FULL COLUMNS` for the MCP table-schema path - preserve existing safe two-part and catalog-qualified identifier handling - map Doris column comments into the existing `comment` response field - add unit coverage for internal and external catalogs - add a real Doris regression through HTTP and subprocess stdio transports ## Root cause The active `get_table_schema` MCP path queried Doris with `DESCRIBE` but then attempted to read a `Comment` field. Doris `DESCRIBE` does not return that field, so column comments were always empty even when they existed. `SHOW FULL COLUMNS` returns the same schema fields used by the current response mapping plus the actual `Comment` value. ## Validation - `uv run pytest -q`: 1088 passed, 68 skipped; coverage 57.95% - focused metadata tests: 41 passed - real Doris HTTP and subprocess stdio suite: 9 passed, 2 optional monitoring tests skipped - real Doris regression creates a commented column and verifies the MCP result - `uv run ruff check .` - `uv run mypy doris_mcp_server` - CI-equivalent Bandit scan Fixes #81 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
