• Stan Hu's avatar
    Reduce queries needed to check if node is a primary or secondary Geo node · 6ec3953e
    Stan Hu authored
    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:
    
    ```sql
    SELECT  1 AS one FROM "geo_nodes" LIMIT 1
    ```
    
    This commit fixes the caching to handle boolean values properly.
    
    Closes #1644
    6ec3953e
geo_spec.rb 2.83 KB