michael-s-molina commented on PR #32520: URL: https://github.com/apache/superset/pull/32520#issuecomment-2733580913
Thanks for the PR @betodealmeida. Some thoughts: > Note that for metrics and columns type and name are redundant (since they're still present in the response payload under metrics and columns, respectively). But to make the API clearer I uncluded them in the response, even if technically only the UUID is needed. This might be problematic for us given that we have a dataset with 30k metrics and 7k columns and this decision would significantly increase the payload size. I think we'll need a very compact API response and a very efficient algorithm when changing the order of things considering the volume of metrics/dimensions. cc @justinpark @villebro -- 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]
