Scott Ganyo

Results 49 comments of 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...