bito-code-review[bot] commented on PR #43652: URL: https://github.com/apache/superset/pull/43652#issuecomment-5457555308
<!-- Bito Reply --> The issue arises because adding the `clickhouse` extra to `requirements/development.in` triggers the installation of `clickhouse-connect`. When `superset.db_engine_specs.clickhouse` is imported, it likely attempts to access `app.config` at the module level, which fails outside of a Flask application context. To resolve this, you can either move the configuration access inside a function or method that is only called when the application is active, or use a lazy-loading pattern for the configuration values. Alternatively, if the dependency is only needed for CI, you could consider moving it to a separate CI-specific requirements file, though making the module-level code app-context safe is generally the preferred approach for library stability. -- 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]
