codeant-ai-for-open-source[bot] commented on PR #38689:
URL: https://github.com/apache/superset/pull/38689#issuecomment-4073046489

   ## **Sequence Diagram**
   
   This PR updates query building so real column names are always the primary 
mapping keys, while verbose names are added only as non-conflicting aliases. 
This keeps filter resolution predictable and prevents verbose labels from 
overriding actual SQL column identifiers.
   
   ```mermaid
   sequenceDiagram
       participant Client
       participant QueryBuilder
       participant ColumnMap
       participant SQLBuilder
   
       Client->>QueryBuilder: Build query with filter field
       QueryBuilder->>ColumnMap: Register each column_name as primary key
       QueryBuilder->>ColumnMap: Add verbose_name only when key is unused
       QueryBuilder->>ColumnMap: Resolve filter field to mapped column
       QueryBuilder->>SQLBuilder: Build filter using resolved real column
       SQLBuilder-->>Client: Return query with correct filtering
   ```
   
   ---
   *Generated by [CodeAnt AI](https://codeant.ai)*
   


-- 
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]

Reply via email to