search

apache / superset

Apache Superset is a Data Visualization and Data Exploration Platform
https://superset.apache.org/
Apache License 2.0
60.61k stars 13.1k forks source link

Alerts .csv does not use a chart's time filters -- regression in 3.0.0 #25538

Open sfirke opened 9 months ago

sfirke commented 9 months ago

Summary: after upgrading from 2.1.0 to 3.0.0, a table chart with a time column filter fails to have that filter applied when sent via Alerts & Reports as a .csv attachment or plain text. When table is sent as an image, the chart is filtered correctly.

The filter applying when the chart is sent as an image should hopefully be a clue as to where the problem lies.

Under 2.1.0 The alert sends a .csv with 14 records, as the chart has this filter for last 30 days: image

Under 3.0.0 The alert sends a .csv with 219 records, despite the chart itself correctly showing only 14 records. A SQL query confirms that 219 is the total # of records in the table when no time filter is applied.

bkyryliuk commented 4 months ago

We are experiencing the same issue on v 3.1.0

rusackas commented 4 weeks ago

Hey folks, this issue has gone stale, but at least it was reported in a current version. Is it by any luck fixed in 4.x.x? If nobody wants to tackle it, it might be closed before too long.

sfirke commented 4 weeks ago

I tried to validate it but I ran into a different bug where I'm not able to send .csv attachments for scheduled chart reports: #28595 So I can't validate this either way right now 😩