rusackas commented on PR #41714: URL: https://github.com/apache/superset/pull/41714#issuecomment-5196666046
@sancho11 this is exactly the kind of testing this migration needs before it lands, thanks for tracing all this. 1 and 2 were already fixed on the branch, autozoom buckets by slice_id and `enableNoResults: false` skips the empty-state. Just pushed a fix for 4, switching to `result_type: 'results'` so it skips the async handoff you traced. 3 is real too, and the trickiest of the four, a new endpoint scoped to the container's own access rather than the per-chart check. Want to open that one against this branch like you offered? I'd rather have your production traces backing it than guess at it myself. -- 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]
