Erik Ritter
Erik Ritter
Does the `query key` you discuss here differ from the chart `cache_key`? Can we reuse the cache key logic here?
Nope, that makes sense to me! Simply wanted to make sure that you knew we already had logic to generate this key that could be reused for this use case
Closing as stale, please file a new issue if you're still seeing this
Looks like this is completed in #417 so i'm going to close
Tagging @ktmud and linking to some of his thoughts on this topic from a PR comment: https://github.com/apache/incubator-superset/pull/10274#discussion_r454127147
Paying this forward, I took a trip through multiple github issues/repos and finally found https://github.com/nedbat/coveragepy/issues/841#issuecomment-1170098080 which fixed coverage for projects that used async sqlalchemy. Adding: ``` [tool.coverage.run] concurrency = ["greenlet"]...