feat: Read from replicas in caches#3383
Conversation
ruslandoga
force-pushed
the
rd/read-from-replicas-in-caches
branch
3 times, most recently
from
49bb31f to
44c5cb2
Compare
ruslandoga
force-pushed
the
rd/read-from-replicas-in-caches
branch
from
44c5cb2 to
3ef1610
Compare
| distribution("logflare.repo.query.total_time", | ||
| unit: {:native, :millisecond}, | ||
| tags: [:role], | ||
| tag_values: &add_repo_role/1 |
There was a problem hiding this comment.
Not sure if this is needed at this stage.
There was a problem hiding this comment.
Removed this role tag from all metrics in 03ac56f
| end | ||
|
|
||
| defp random_read_replica do | ||
| case fetch_read_replicas!() do |
There was a problem hiding this comment.
We want to load balance across all including the primary for first pass at this.
There was a problem hiding this comment.
Added primary to the set of choices in 03ac56f
| distribution("logflare.repo.query.total_time", | ||
| unit: {:native, :millisecond}, | ||
| tags: [:role], | ||
| tag_values: &add_repo_role/1 |
| unit: {:native, :millisecond}, | ||
| tags: [:role], | ||
| tag_values: &add_repo_role/1 | ||
| ) |
|
Also needs documentation updates for self hosting |
I've added some documentation in 674ee36. Off-topic: there is no description on that page for most of the other database configuration options like |
|
@ruslandoga can you add the remaining missing db env vars in a follow up PR? 🙏 |
2 participants
Closes O11Y-1354