posthog icon indicating copy to clipboard operation
posthog copied to clipboard

fix(insights): Use lru_cache to reduce queries to instance settings

Open webjunkie opened this issue 1 year ago • 3 comments

Problem

Lots of queries to instance settings

Screenshot 2024-04-29 at 14 32 15

Changes

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Does this work well for both Cloud and self-hosted?

How did you test this code?

webjunkie avatar Apr 29 '24 12:04 webjunkie

🔍 Existing Issues For Review

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

📄 File: posthog/models/instance_setting.py

Function Unhandled Issue
value JSONDecodeError: Expecting value: line 1 column 1 (char 0) /admin/posthog/instan...
Event Count: 1

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

sentry[bot] avatar Apr 29 '24 12:04 sentry[bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.

posthog-bot avatar May 07 '24 07:05 posthog-bot

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.

posthog-bot avatar May 15 '24 07:05 posthog-bot

This PR was closed due to lack of activity. Feel free to reopen if it's still relevant.

posthog-bot avatar May 23 '24 07:05 posthog-bot