dosubot[bot] commented on issue #30250:
URL: https://github.com/apache/superset/issues/30250#issuecomment-2345942869

   <!-- Greeting -->
   Hey @avalanchecrusader! I'm a bot designed to help you with bugs, questions, 
and contributions. I'm here to assist you while you wait for a human 
maintainer. Let's tackle this Superset issue together!
   
   <!-- Answer -->
   I found a similar closed issue: [Superset showing offline or doesn't 
load](https://github.com/apache/superset/issues/21808), which might provide 
some insights <sup>[[1]](https://github.com/apache/superset/issues/21808)</sup>.
   
   Regarding your specific problem, the collapse during the initialization of a 
dashboard with multiple filters and elements after migrating the database from 
GBQ to Trino is likely due to the initialization logic in the 
`useInitialization` hook. The initialization process depends on the number of 
charts that are not in the 'loading' state:
   
   ```ts
   const numberOfLoadedCharts = Object.values(charts).filter(
     ({ chartStatus }) => chartStatus !== 'loading',
   ).length;
   if (
     numberOfLoadingCharts > 0 &&
     numberOfLoadedCharts >= numberOfLoadingCharts
   ) {
     setIsInitialized(true);
   }
   ```
   
   If the charts are not loading correctly or taking too long to load due to 
the database migration, the initialization process may not complete as 
expected, causing the dashboard to collapse. The migration to Trino might have 
introduced performance issues or compatibility problems that affect how charts 
are loaded and rendered.
   
   Additionally, the logic to set `isInitialized` to `true` if there are no 
charts to load after a 1-second timeout might not be sufficient if the charts 
are taking longer to load due to the new database backend:
   
   ```ts
   if (numberOfLoadingCharts === 0) {
     filterTimeout = setTimeout(() => {
       setIsInitialized(true);
     }, 1000);
   }
   ```
   
   To address this issue, you may need to investigate the performance and 
compatibility of Trino with your Superset setup, ensuring that the charts can 
load within the expected time frame 
<sup>[[2]](https://github.com/apache/superset/blob/master/superset-frontend/src/dashboard/components/nativeFilters/FilterBar/state.ts)</sup>.
   
   
   <!-- Dosu Comment Footer -->
   *To continue talking to [Dosu](https://dosu.dev), mention @dosu.*
   


-- 
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: notifications-unsubscr...@superset.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to