graceguo-supercat opened a new pull request #9060: [domain sharding] Freeup main domain when domain sharding is enabled URL: https://github.com/apache/incubator-superset/pull/9060 ### CATEGORY Choose one - [x] Bug Fix - [x] Enhancement (new features, refinement) - [ ] Refactor - [ ] Add tests - [ ] Build / Development Environment - [ ] Documentation ### SUMMARY When user opens a large dashboard, it may trigger a large amount of data request to backend. Most browser only allow 6 concurrent requests to the same domain. So we use sub-domains (and domain sharding) in airbnb to allow more concurrent requests. If the chart data is well cached, the result of domain-sharding is very positive. When user open a large dashboard and its content is not well cached, all the slots are blocked by slow queries. At the same time if user tries to make API call to backend, there will be no response in the page. for example, check/uncheck fav star, save dashboard edit etc. This make user confused and feel dashboard is broken. If we enable domain sharding, we can allocate a domain for user events. This PR is to free the main domain away from chart loading, so that user interactivity won't be blocked by slow queries. ### TEST PLAN added unit test ### REVIEWERS @etr2460 @robdiciuccio @mistercrunch
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
