[GitHub] [incubator-superset] robdiciuccio commented on issue #9190: [SIP-39] Global Async Query Support

[GitBox <gi...@apache.org>]

robdiciuccio commented on issue #9190: [SIP-39] Global Async Query Support
URL: https://github.com/apache/incubator-superset/issues/9190#issuecomment-594242375
 
 
   @DiggidyDave The use cases for long-running queries in SQL Lab and loading dashboards in a performant manner feel like very different use cases. Polling could (and currently does) satisfy the SQL Lab use case, though there is significant room for improvement there. The latency introduced by polling in dashboards would be a much larger impact to user experience.
   
   Agreed that this discussion is healthy! Architecture proposals should be actively debated and scrutinized so we all benefit from better informed decision-making.
   
   @williaster `@superset-ui` will need to be upgraded to also support [WebSocket](https://developer.mozilla.org/en-US/docs/Web/API/WebSocket) connections. This upgrade should be backwards compatible and configured via feature flags. [CORS does not apply to WebSockets](https://blog.securityevaluators.com/websockets-not-bound-by-cors-does-this-mean-2e7819374acc), but securing the WS connections by validating origin domain is recommended, so we should keep the cross-domain use case in mind during implementation of these security restrictions.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org