Let's delete this line

serverThreadInfo is an array. We typically try to use plurals for arrays. The English word "information" is technically both singular and plural, but to disambiguate in the code we generally try to use the suffix "Infos"

The convention in the codebase is to omit semicolons at the end of SQL statements unless using multipleStatements: true

It's best to try to match conventions as much as possible, rather than changing things to match your preference. Please try to identify conventions on your own, and try to match the style in the codebase as much as possible!

Nit: convention in the code is to use capital letter for the second letter of ID

Nit

I don't think we need this condition here, as it doesn't appear to be used in your use case. It's best to keep things as simple as possible, and avoid "overengineering" for potential future use cases. If we need this in the future, it should be really easy to reintroduce it

I don't think this is necessary. We typically let DB errors surface directly... they're helpful to see in the keyserver log because of this line. Meanwhile, this line makes sure that we don't surface anything weird to the API caller