codeant-ai-for-open-source[bot] commented on PR #38676:
URL: https://github.com/apache/superset/pull/38676#issuecomment-4118475620
## **Sequence Diagram**
This PR changes native filter generation to return both filter config and an
optional warning. When a filter type is unrecognized, the filter is skipped and
a warning is propagated so it can be shown in the report execution log.
```mermaid
sequenceDiagram
participant ReportJob
participant ReportSchedule
participant FilterBuilder
participant ExecutionLog
ReportJob->>ReportSchedule: Build native filter params
loop For each native filter
ReportSchedule->>FilterBuilder: Generate filter from filter type
alt Filter type recognized
FilterBuilder-->>ReportSchedule: Return filter config
ReportSchedule->>ReportSchedule: Merge config into params
else Filter type unrecognized
FilterBuilder-->>ReportSchedule: Return empty config and warning
ReportSchedule->>ReportSchedule: Skip filter and collect warning
end
end
ReportSchedule-->>ReportJob: Return params and warnings
ReportJob->>ExecutionLog: Add warnings to report execution log
```
---
*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]
