aminghadersohi opened a new pull request, #41594: URL: https://github.com/apache/superset/pull/41594
### SUMMARY Research spike mapping out what "Semantic Layer MCP" means for MCP v2, what Superset already covers, and what needs to be built. Adds `RESEARCH.md` with: - Analysis of Superset's two semantic layer systems (built-in `SqlaTable` + external `SemanticLayer/SemanticView`) - Inventory of existing MCP v1 tool coverage - Q&A with betodealmeida on open design questions (answered 2026-06-04) - **Final architecture proposal** based on Beto's answers **Key architectural decisions documented:** The intended LLM workflow is a flat metric-first pattern (not a hierarchical layer-first discovery flow): 1. `list_metrics` — unified search across built-in + external layers, includes compatible dimensions per metric inline 2. `get_table(metrics, dimensions, filters)` — unified query tool routing to built-in or external path New tools proposed (in priority order): | Tool | Phase | Notes | |---|---|---| | `list_metrics` | 1 | Primary entry point; spans SqlMetric + SemanticView; no layer selection required | | `get_table` | 1 | Unified query; replaces separate `query_dataset` + `query_semantic_view` for semantic workflows | | `get_compatible_dimensions` | 1 | Surfaces `view.get_compatible_dimensions()` from `semantic_layers/view.py` | | `get_compatible_metrics` | 1 | Surfaces `view.get_compatible_metrics()` | | `get_dimension_values` | 2 | Surfaces `view.get_values()` — needs Snowflake plugin | | `get_row_count` | 2 | Surfaces `view.get_row_count()` — needs Snowflake plugin | | `list_semantic_layers` | 3 | Admin discovery | | `get_semantic_layer_info` | 3 | Admin discovery | | `list_semantic_views` | 3 | Admin discovery | Total estimated effort: ~4 weeks across 3 phases. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF N/A — documentation only. ### TESTING INSTRUCTIONS No code changes. Review `RESEARCH.md` for accuracy of the architecture proposal. ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
