Return ERROR on COMMIT instead of silently ROLLBACK

Edit
Comment/Review
Change Status
ID 2522
Title Return ERROR on COMMIT instead of silently ROLLBACK
Topic SQL Commands
Created 2020-03-31 18:19:04
Last modified 2021-03-25 19:24:01 (3 years, 8 months ago)
Latest email 2021-03-25 19:23:53 (3 years, 8 months ago)
Status
2021-03: Returned with feedback
2021-01: Moved to next CF
2020-11: Moved to next CF
2020-09: Moved to next CF
2020-07: Moved to next CF
Target version
Authors Dave Cramer (dcramer)
Reviewers Laurenz Albe (laurenz)Become reviewer
Committer
Links CFbot results (CirrusCI) CFbot GitHub
Checkout latest CFbot patchset Go to your local checkout of the PostgreSQL repository and run: 
git remote add commitfest https://github.com/postgresql-cfbot/postgresql.git
git fetch commitfest cf/2522
git checkout commitfest/cf/2522
Emails
Error on failed COMMIT
First at 2020-02-11 21:44:50 by Vik Fearing <vik at postgresfriends.org>
Latest at 2021-03-25 19:23:53 by David Steele <david at pgmasters.net>
Latest attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2021-01-25 16:29:10 from Dave Cramer <davecramer at postgres.rocks>
    Attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2021-01-25 16:29:10 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2021-01-25 14:09:09 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2020-11-10 16:53:20 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2020-11-09 21:26:59 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (0001-Throw-error-and-rollback-on-a-failed-transaction-ins.patch) at 2020-08-04 16:19:27 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (throwerror2.patch) at 2020-02-27 16:30:17 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (throwerror2.patch) at 2020-02-27 12:44:14 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
    Attachment (0001-change-commit-semantics-to-throw-an-error-and-then-r.patch) at 2020-02-26 20:51:39 from Dave Cramer <davecramer at postgres.rocks> (Patch: Yes)
History
When Who What
2021-03-25 19:24:01 David Steele (dsteele) Closed in commitfest 2021-03 with status: Returned with feedback
2021-03-25 16:04:56 David Steele (dsteele) New status: Waiting on Author
2021-02-01 12:48:31 Masahiko Sawada (masahikosawada) Closed in commitfest 2021-01 with status: Moved to next CF
2021-01-28 11:01:09 Masahiko Sawada (masahikosawada) New status: Needs review
2021-01-26 10:06:12 Laurenz Albe (laurenz) Added laurenz as reviewer
2020-12-01 09:49:04 Georgios Kokolatos (gkokolatos) New status: Waiting on Author
2020-12-01 09:48:16 Georgios Kokolatos (gkokolatos) Closed in commitfest 2020-11 with status: Moved to next CF
2020-12-01 09:48:08 Georgios Kokolatos (gkokolatos) Posted comment with messageid <160681608817.7563.12916589546376627306.pgcf@coridan.postgresql.org>
2020-12-01 09:48:08 Georgios Kokolatos (gkokolatos) New status: Needs review
2020-11-10 15:19:53 Georgios Kokolatos (gkokolatos) Posted comment with messageid <160502159371.7362.5481306175095325118.pgcf@coridan.postgresql.org>
2020-11-10 15:19:53 Georgios Kokolatos (gkokolatos) New status: Waiting on Author
2020-10-30 16:13:13 Georgios Kokolatos (gkokolatos) Posted comment with messageid <160407439378.1204.12914085632812820529.pgcf@coridan.postgresql.org>
2020-09-30 07:20:24 Michael Paquier (michael-kun) Closed in commitfest 2020-09 with status: Moved to next CF
2020-09-30 07:20:20 Michael Paquier (michael-kun) New status: Needs review
2020-09-30 07:19:12 Michael Paquier (michael-kun) Closed in commitfest 2020-09 with status: Returned with feedback
2020-08-03 09:12:29 Daniel Gustafsson (d_gustafsson) New status: Waiting on Author
2020-08-03 09:12:19 Daniel Gustafsson (d_gustafsson) Closed in commitfest 2020-07 with status: Moved to next CF
2020-07-02 13:46:43 Daniel Gustafsson (d_gustafsson) Changed authors to Dave Cramer (dcramer)
2020-04-08 16:16:00 David Steele (dsteele) Changed targetversion to None
2020-03-31 18:20:06 Dave Cramer (dcramer) Changed targetversion to 13
2020-03-31 18:19:05 Dave Cramer (dcramer) Attached mail thread b9fb50dc-0f6e-15fb-6555-8ddb86f4aa71@postgresfriends.org
2020-03-31 18:19:04 Dave Cramer (dcramer) Created patch record
Edit
Comment/Review
Change Status