codeant-ai-for-open-source[bot] commented on PR #38094:
URL: https://github.com/apache/superset/pull/38094#issuecomment-4047374320
## **Sequence Diagram**
This PR changes TimeTable row construction so each row carries raw cell
values for every column, which the sorter now uses directly for the clicked
column. It also preserves null values during rendering so nulls display as null
and sparkline tooltips are not shown for missing points.
```mermaid
sequenceDiagram
participant User
participant TimeTable
participant ValueCalculator
participant TableEngine
participant SortComparator
User->>TimeTable: Open time series table
TimeTable->>ValueCalculator: Compute value for each row and column
ValueCalculator-->>TimeTable: Return cell values map and display content
TimeTable-->>TableEngine: Provide rows with cellValues and rendered cells
User->>TableEngine: Click a metric column header
TableEngine->>SortComparator: Sort rows for selected column
SortComparator-->>TableEngine: Compare using row cellValues for that
column
TableEngine-->>User: Show correctly sorted rows with nulls preserved
```
---
*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]