michael-s-molina commented on PR #20143: URL: https://github.com/apache/superset/pull/20143#issuecomment-1248441353
> Hi @michael-s-molina. Can we rebase it from `master` branch? There's a good idea, use `QUERY MODE` is `Raw Records` in `table` viz and select all columns by default. For example when we click a dataset name in Dataset page then we are redirected to explore page and see `all columns` result after chart execution. Because the vast majority of users use table viz with all columns. What do you think about it? Hi @EugeneTorap. This draft is only meant to be a POC. We just merged https://github.com/apache/superset/pull/21094 and now the next step is to add Select All to the sync version. Async support will come next, in a different PR than this one. I'll close this PR to avoid further confusion. The Select All in Explore is a specific case and there's a discussion about it being conducted by @kasiazjc @cccs-RyanK. Here we're focusing more on the generic feature for all selects. @kasiazjc @cccs-RyanK I would appreciate it if you could post the discussion thread or any doc reference so that @EugeneTorap can participate. -- 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]
