The schema cache load is not retried with exponential backoff #3523
Labels
Comments
This was referenced May 15, 2024
|
Still an issue, this was incorrectly closed by #3536 |
steve-chavez
added a commit
to steve-chavez/postgrest
that referenced
this issue
Jul 11, 2024
Fixes PostgREST#3523. Now if obtaining the pg version or schema cache fails, we do the same retrying process. This way we don't get two retries. Refactors and renames the "connectionWorker" to "schemaCacheLoader". This makes more sense since what we really want is the schema cache, the version is the pre-requisite for ensuring our schema cache queries will work. Additionally, we no longer log ` Attempting to connect to the database...` at startup unnecessarily. This is only logged whenever there's a retry attempt.
steve-chavez
added a commit
to steve-chavez/postgrest
that referenced
this issue
Jul 11, 2024
Fixes PostgREST#3523. Now if there's a failure when obtaining the pg version OR schema cache, we do the same retrying process. This way we don't add two retries. Refactors and renames the "connectionWorker" to "schemaCacheLoader". This makes more sense since what we really want is the schema cache, the version is the pre-requisite for ensuring our schema cache queries work. Additionally, we no longer log ` Attempting to connect to the database...` at startup unnecessarily. This is only logged whenever there's a retry attempt.
steve-chavez
added a commit
that referenced
this issue
Jul 11, 2024
Fixes #3523. Now if there's a failure when obtaining the pg version OR schema cache, we do the same retrying process. This way we don't add two retries. Refactors and renames the "connectionWorker" to "schemaCacheLoader". This makes more sense since what we really want is the schema cache, the version is the pre-requisite for ensuring our schema cache queries work. Additionally, we no longer log ` Attempting to connect to the database...` at startup unnecessarily. This is only logged whenever there's a retry attempt.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
Adding the following on
test/spec/fixtures/schema.sql:Makes the schema cache retry load on an endless loop without backoff:
Previously reported on #3424 (comment)
The text was updated successfully, but these errors were encountered: