Skip to content
Snippets Groups Projects

Reduce queries needed to check if node is a primary or secondary Geo node

Merged Stan Hu requested to merge sh-fix-geo-node-cache into master

If the Geo check for secondary? failed, the RequestStore cache would always be set to false but the cache would not actually be used due to the semantics of the ||= operator. This led to dozens of SQL calls with:

SELECT  1 AS one FROM "geo_nodes" LIMIT 1

This commit fixes the caching to handle boolean values properly.

Closes #1644 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading