giftig commented on issue #25518:
URL: https://github.com/apache/superset/issues/25518#issuecomment-1753214228

   I haven't tried with SQLite, only with postgres; it looks like the update 
error you're seeing there is preventing us from saving the `query_id`, and then 
`query_id` is missing from where it's assumed to be present in 
`sql_lab.get_query_backoff_handler`, so this'll be the root cause. What's 
unclear is why it's attempting to update this from a different thread; the code 
looks like this:
   
   ```python
           execute_thread = threading.Thread(target=_execute, 
args=(execute_result,))
           execute_thread.start()
   
           # Wait for a query ID to be available before handling the cursor, as
           # it's required by that method; it may never become available on 
error.
           while not cursor.query_id and not execute_result.get("complete"):
               time.sleep(0.1)
   
           logger.debug("Query %d: Handling cursor", query.id)
           cls.handle_cursor(cursor, query, session)
   ```
   and only attempts to run the actual `cursor.execute` call in a thread, then 
waits for `query_id` to appear and continues in the main thread. Hopefully we 
can dig out more information by looking at the full stack trace of the 
underlying query failure.


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