fix(plugin-server): Remove Postgres-based plugin error logging in favor of existing ClickHouse-based approaches#18764

Merged

tkaemming merged 25 commits intomasterfrom remove-plugin-error-update

Nov 27, 2023

+227-255

Commits on Nov 20, 2023

Check if we need to flush after queueing, rather than before -- this improves the chances we'll hit our delivery targets (and also makes this easier to test.)
tkaemming
committed
Remove UPDATEs from error callback.
tkaemming
committed
Hack together an ugly test that passes, but requires running server with APP_METRICS_FLUSH_FREQUENCY_MS=0.
tkaemming
committed
Move error parsing down into helper function.
tkaemming
committed
Update the remainder of the tests (...which error?)
tkaemming
committed
Remove a case so nice it was tested twice...
tkaemming
committed
Also remove unnecessary chaining from unhandled promise rejection.
tkaemming
committed
Check for log entry, not plugin error.
tkaemming
committed
Clean up some messy wording.
tkaemming
committed
Set a reminder for what actually needs to be cleaned up with types here.
tkaemming
committed

Commits on Nov 23, 2023

Commits on Nov 27, 2023