kgabryje edited a comment on pull request #19064: URL: https://github.com/apache/superset/pull/19064#issuecomment-1062788333
> @kgabryje Thank you for the perf improvement! just 1 question here: I saw both `getChartIdsInFilterBoxScope` and `getChartIdsInFilterScope` functions. So when to use each of them? @graceguo-supercat I kept `getChartIdsInFilterBoxScope` for legacy reasons. Refactoring the logic related to filter box would be rather risky - I decided not to do it since we're on the road of deprecating filter box in favour in native filters. I added a comment to `getChartIdsInFilterBoxScope` that redirects to the new `getChartIdsInFilterScope`. So going forward, we should use only `getChartIdsInFilterScope` -- 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]
