michael-s-molina commented on issue #33152:
URL: https://github.com/apache/superset/issues/33152#issuecomment-2904117799

   Thanks for writing down the problem statement in detail @mistercrunch. 
   
   > UPDATE chart SET query_context = NULL WHERE chart_id IN (SELECT DISTINCT 
chart_id FROM reports)
   
   This would not resolve one of the main problems which is cache warmup.
   
   We're almost finishing the following option to unblock 5.0:
   
   > write database migrations for query_context
   
   This is not a long term solution and we definitely need something better for 
the project. @villebro and I were planning on tackling this problem while 
transforming charts into extensions but this might take a while. We could 
introduce an endpoint to regenerate the query_context but I don't know how that 
would be invoked from migrations.
   
   We tested the [query_context migration with 
Python](https://github.com/apache/superset/pull/33285) using 50k charts and  
compared with the Javascript counterpart achieving 100% match. This should be 
sufficient to unblock 5.0 and give us time to think about a long term solution.


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

Reply via email to