aminghadersohi commented on code in PR #44393:
URL: https://github.com/apache/superset/pull/44393#discussion_r4250811077
##########
docs/admin_docs/configuration/mcp-server.mdx:
##########
@@ -327,6 +327,46 @@ Permissions and row-level security apply exactly as they
do without it.
---
+## Semantic-view chart targets
+
+`generate_chart` and `generate_explore_link` require exactly one source
selector:
+`dataset_id` for a table dataset, or `view_id` for a semantic view. View
selectors
+accept UUIDs or legacy numeric IDs. The numeric ID spaces are independent: a
+dataset ID must not be reused as a view ID.
+
+The legacy `datasource_id` alias defaults to a table when `datasource_type` is
+omitted. With `datasource_type: "semantic_view"`, it selects a view instead.
+For table-only requests, an explicit `dataset_id` takes precedence over
+`datasource_id`. Unsupported types and conflicting semantic selectors are
rejected.
+
+Semantic configurations use saved metric names, for example
+`{"name": "revenue", "saved_metric": true}`, and dimensions published by the
view.
+Ad-hoc aggregates and SQL expressions are rejected. Missing and inaccessible
+views return the same `view_not_found` error without exposing view metadata.
+
+`update_chart` accepts at most one replacement selector; omitting both retains
+the chart's source. A `view_id` rebind validates retained query roles; a
+`dataset_id` rebind, including from a view, requires a complete `config`.
Review Comment:
Our earlier wording here was incomplete: Gauge, Treemap and geographic
charts reject any config-less rebind, `view_id` included
(`requires_config_for_dataset_rebind` in `update_chart`).
```suggestion
the chart's source. A `view_id` rebind validates retained query roles, except
Gauge, Treemap and geographic charts, which require a complete `config` for
any
rebind. A `dataset_id` rebind, including from a view, requires a complete
`config`.
```
--
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]
