Scott Ganyo
Scott Ganyo
I think this would need to be determined in the context of SLO and performance requirements.
I'd suggest just having a function called "SetConfig()" that sets the config pointer and call it a config, not a cache. I don't believe we need the useCache or clearConfig...
I see... So I think what I hear you saying is that you'd like to load and keep certain external values (from, I assume, env vars), but then override other...
Counter-point: CI did its job of pointing out an issue that we might not have noticed until a user attempted an install of Postgres 15. This change will likewise ensure...
Counter counter-point: Only testing on latest could cause us to accidentally drop support for older versions. Arguably, we should: 1. Test on each platform version that we support. 2. Also...
Is this perhaps an issue with how we are doing retries?
I agree, we should ensure the server will connect and reconnect to the db appropriately, then let the server start but fail all requests with a 503 Service Unavailable error...
One potential race condition is when requesting an update w/ `req.GetAllowMissing = true` when the entity doesn't exist in the database. In this case, the action will attempt: 1. Update...
Llooking more closely at the [grpc status codes guide](https://grpc.github.io/grpc/core/md_doc_statuscodes.html), it looks like for this case, we could / should simply translate the error to ABORTED. I'll open a ticket for...
I assume this applies higher in the path hierarchy as well? So taking the above heuristic, I might expect that this path: `projects/demo/locations/global/versions/v1/specs/-/artifacts/-` would obey the above rule: "A list...