posthog icon indicating copy to clipboard operation
posthog copied to clipboard

fix(hogql): exclude exports from redis cache

Open thmsobrmlr opened this issue 1 year ago • 1 comments

Problem

We're caching HogQL exports, but that's not really necessary.

Changes

Prevents caching the exports.

How did you test this code?

Tried locally

thmsobrmlr avatar May 14 '24 14:05 thmsobrmlr

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: posthog/hogql_queries/query_runner.py

Function Unhandled Issue
run JSONDecodeError: Input must be bytes, bytearray, memoryview, or str: line 1 column 1 (char 0) ...
Event Count: 1.4k
run KeyError: 'cached_response' posthog.tasks.tasks.p...
Event Count: 103
run ValidationError: 1 validation error for HogQLQueryResponse posthog.tasks.tasks.process_...
Event Count: 67
run TypeError: a bytes-like object is required, not 'CachedTrendsQueryResponse' posthog.tas...
Event Count: 57
run TypeError: Integer exceeds 64-bit range posthog.t...
Event Count: 49

Did you find this useful? React with a 👍 or 👎

sentry-io[bot] avatar May 14 '24 14:05 sentry-io[bot]