kgabryje commented on PR #38015:
URL: https://github.com/apache/superset/pull/38015#issuecomment-3915476020
> 1. I believe the solution here is just removing it from the
output/response, but not really excluding it from the actual DB query
thumbnail_url is a computed property, we don't query it from db. The perf
issue originated from the expensive get_dashboard_digest() call, which we now
avoid
> 2. @dpgaspar question for you: native support for select_columns / columns
isn't available in FAB yet for /api/v1/{model}/{id}, right? Do you think it
makes more sense if this logic lives there?
Yeah I think native support for column filtering in FAB would be better long
term! But also curious to hear Daniel's opinion
--
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]
