mistercrunch commented on issue #8655: [SIP-30] Remove Tabs in SQL Lab
URL: 
https://github.com/apache/incubator-superset/issues/8655#issuecomment-571826828
 
 
   About change management, we did something similar for both "explore v2" and 
"dashboard v2", where in the case of "dashboard v2", the dashboards had to be 
validated (and in some cases tweaked) manually individually.
   
   Things can get intricate as components, endpoints and objects may become 
shared in intricate ways. For example, the work of deprecating "explore v1" 
became fairly intricate as it was sharing things with v2. Clearly there are 
pros and cons here, and if we decide to have 2 versions overlap, we should be 
deliberate on whether we want to go with a "not DRY" (where code is duplicated 
to evolve independently) or a tangled approach (where both versions share 
components and objects).
   
   One thing to mention is that the design is not very clear around state 
management, and making sure people don't loose work. Personally I'd push 
towards a "Google Docs"-type pattern, forcing people to title the session 
quickly, and autosaving as the query is executed. We want to make sure that 
whatever happens to the browser tab, no work will be lost. We should have clear 
answers to "what happens if I have 6 tabs open, close the browser, and want to 
get back to some of the tabs that were opened before? what if I had not 
explicitly saved? is a tab really a query session? can I restore the session if 
I close the tab, or just the latest state?

----------------------------------------------------------------
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]

Reply via email to