villebro commented on PR #30438:
URL: https://github.com/apache/superset/pull/30438#issuecomment-2427713160

   > > @Mainer-g00t @fmannhardt as you raised concerns here, did you find that 
this PR broke your use cases?
   > 
   > Yes, it does. In our case, charts export filters that are picked up by 
jinja logic to filter dataset with data that is not part of the columns in the 
dataset. Specifically, our dataset looks like:
   > 
   > * customer (1k), products (1k), providers(1k), metrics...
   > 
   > This leads to a roughly 500M row table. We split it in 3 datasets, one for 
each dimension and use x-filters to select on the other datasets. ie, clicking 
on one customer on the first table will filter all products and providers for 
that customer and so on.
   > 
   > This allows us to display relationships in big datasets breaking it up in 
dimensions and using x-filters
   
   Just to confirm, have you tested the changes here and can confirm that they 
do indeed break your use case? Because to me it seems like you should still be 
able to do the same cross filtering even after this change, as long as you've 
defined your filter scopes correctly.


-- 
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]

Reply via email to