fix: do not retry transactions on any error#4222
fix: do not retry transactions on any error#4222laurenceisla wants to merge 6 commits intoPostgREST:mainfrom
Conversation
1afa5fb to
665f55c
Compare
|
Should we vendor hasql-notifications for now since there's no response for diogob/hasql-notifications#30 ? I remember we did this before too. It's a single file so no major maintenance problem there IMO. |
|
We should not vendor it, for two reasons:
|
510cafa to
ff133bf
Compare
This needs more involved changes in the hsie source code, which can be deferred to later.
ff133bf to
23426d2
Compare
Thanks for the ping suggestion (I should've done this weeks ago). Changes are now merged and released upstream!
OK, so this would be the only blocker for this PR for now then. Also some |
WIP
Should close #3673.
hasqlversion: Add support for hasql 1.9 diogob/hasql-notifications#30