yousoph commented on code in PR #41462:
URL: https://github.com/apache/superset/pull/41462#discussion_r3494771625
##########
superset/models/helpers.py:
##########
@@ -3350,6 +3350,23 @@ def get_sqla_query( # pylint:
disable=too-many-arguments,too-many-locals,too-ma
select_exprs.append(outer)
elif columns:
for selected in columns:
+ # Resolve a known column directly to its ``TableColumn`` so
that
+ # multi-part identifiers (e.g. a BigQuery STRUCT field
registered
+ # with a dotted ``column_name`` such as ``a.b.c``) are quoted
per
+ # segment, matching the chart/groupby selection path above.
+ # Routing these through ``quote()`` + sqlglot normalization
below
+ # can collapse the dotted path into a single quoted identifier
+ # (e.g. ```a.b`.`c```), which breaks drill to detail / samples
+ # queries on nested columns (SC-111745).
Review Comment:
Good catch — fixed in 844695acb0. Reworded the comment to describe the
merged single-identifier form unambiguously instead of the confusing `a.b`.`c`
example.
--
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]
