MIME-Version: 1.0
From: Pavel Borisov <pashkin.elfe@gmail.com>
Date: Tue, 24 Feb 2026 20:13:02 +0400
Message-ID: <CALT9ZEHcE6tpvumScYPO6pGk_ASjTjWojLkodHnk33dvRPHXVw@mail.gmail.com>
Subject: New isolation test insert-conflict-do-update-4 outputs rows in
 alternative ordering
To: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: multipart/mixed; boundary="00000000000036610a064b942a1c"
Archived-At: <https://www.postgresql.org/message-id/CALT9ZEHcE6tpvumScYPO6pGk_ASjTjWojLkodHnk33dvRPHXVw%40mail.gmail.com>
Precedence: bulk

--00000000000036610a064b942a1c
Content-Type: text/plain; charset="UTF-8"

Hi, hackers!

New isolation test insert-conflict-do-update-4 was added by commit 4b760a181ab2

Unlike the previous test (insert-conflict-do-update-3) it doesn't
contain an ordering clause, and when run on a table with a custom
access method, it outputs results in a reverse order.

It's a very minor issue, and the proposed fix is just adding the
missing ORDER BY clause.

Other insert-conflict-do-* isolation tests output just one row except
the mentioned insert-conflict-do-update-3 and
insert-conflict-do-update-4.

The patch is attached

Kind regards,
Pavel Borisov
Supabase

--00000000000036610a064b942a1c
Content-Type: application/octet-stream; 
	name="v1-0001-Fix-sorting-of-output-rows-in-a-new-isolation-tes.patch"
Content-Disposition: attachment; 
	filename="v1-0001-Fix-sorting-of-output-rows-in-a-new-isolation-tes.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mm0svtuf0>
X-Attachment-Id: f_mm0svtuf0

RnJvbSAwZDI3ZTE1MDVjMTAwOWUyZjRhZDBjNDgyYWIyOGQ4M2ZmNGE3NmQ5IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBQYXZlbCBCb3Jpc292IDxwYXNoa2luLmVsZmVAZ21haWwuY29t
PgpEYXRlOiBUdWUsIDI0IEZlYiAyMDI2IDIwOjAwOjQ4ICswNDAwClN1YmplY3Q6IFtQQVRDSCB2
MV0gRml4IHNvcnRpbmcgb2Ygb3V0cHV0IHJvd3MgaW4gYSBuZXcgaXNvbGF0aW9uIHRlc3QKIGlu
dHJvZHVjZWQgaW4gY29tbWl0IDRiNzYwYTE4MWFiMgoKLS0tCiAuLi4vaXNvbGF0aW9uL2V4cGVj
dGVkL2luc2VydC1jb25mbGljdC1kby11cGRhdGUtNC5vdXQgICAgfCA4ICsrKystLS0tCiBzcmMv
dGVzdC9pc29sYXRpb24vc3BlY3MvaW5zZXJ0LWNvbmZsaWN0LWRvLXVwZGF0ZS00LnNwZWMgfCAy
ICstCiAyIGZpbGVzIGNoYW5nZWQsIDUgaW5zZXJ0aW9ucygrKSwgNSBkZWxldGlvbnMoLSkKCmRp
ZmYgLS1naXQgYS9zcmMvdGVzdC9pc29sYXRpb24vZXhwZWN0ZWQvaW5zZXJ0LWNvbmZsaWN0LWRv
LXVwZGF0ZS00Lm91dCBiL3NyYy90ZXN0L2lzb2xhdGlvbi9leHBlY3RlZC9pbnNlcnQtY29uZmxp
Y3QtZG8tdXBkYXRlLTQub3V0CmluZGV4IDZlOTZlMGQxMmRhLi44MGFmMjc5OGE5NCAxMDA2NDQK
LS0tIGEvc3JjL3Rlc3QvaXNvbGF0aW9uL2V4cGVjdGVkL2luc2VydC1jb25mbGljdC1kby11cGRh
dGUtNC5vdXQKKysrIGIvc3JjL3Rlc3QvaXNvbGF0aW9uL2V4cGVjdGVkL2luc2VydC1jb25mbGlj
dC1kby11cGRhdGUtNC5vdXQKQEAgLTEyLDExICsxMiwxMSBAQCBzdGVwIGluc2VydDE6IElOU0VS
VCBJTlRPIHVwc2VydCBWQUxVRVMgKDEsIDExLCAxMTEpCiBzdGVwIHVwZGF0ZTJhOiBVUERBVEUg
dXBzZXJ0IFNFVCBpID0gaSArIDEwIFdIRVJFIGkgPSAxOwogc3RlcCBjMjogQ09NTUlUOwogc3Rl
cCBpbnNlcnQxOiA8Li4uIGNvbXBsZXRlZD4KLXN0ZXAgc2VsZWN0MTogU0VMRUNUICogRlJPTSB1
cHNlcnQ7CitzdGVwIHNlbGVjdDE6IFNFTEVDVCAqIEZST00gdXBzZXJ0IE9SREVSIEJZIGk7CiAg
aXwganwgIGsKIC0tKy0tKy0tLQotMTF8MTB8MTAwCiAgMXwxMXwxMTEKKzExfDEwfDEwMAogKDIg
cm93cykKIAogc3RlcCBjMTogQ09NTUlUOwpAQCAtMzMsNyArMzMsNyBAQCBzdGVwIGluc2VydDE6
IElOU0VSVCBJTlRPIHVwc2VydCBWQUxVRVMgKDEsIDExLCAxMTEpCiBzdGVwIHVwZGF0ZTJiOiBV
UERBVEUgdXBzZXJ0IFNFVCBpID0gaSArIDE1MCBXSEVSRSBpID0gMTsKIHN0ZXAgYzI6IENPTU1J
VDsKIHN0ZXAgaW5zZXJ0MTogPC4uLiBjb21wbGV0ZWQ+Ci1zdGVwIHNlbGVjdDE6IFNFTEVDVCAq
IEZST00gdXBzZXJ0Oworc3RlcCBzZWxlY3QxOiBTRUxFQ1QgKiBGUk9NIHVwc2VydCBPUkRFUiBC
WSBpOwogICBpfCBqfCAgawogLS0tKy0tKy0tLQogICAxfDExfDExMQpAQCAtNTQsNyArNTQsNyBA
QCBzdGVwIGluc2VydDE6IElOU0VSVCBJTlRPIHVwc2VydCBWQUxVRVMgKDEsIDExLCAxMTEpCiBz
dGVwIGRlbGV0ZTI6IERFTEVURSBGUk9NIHVwc2VydCBXSEVSRSBpID0gMTsKIHN0ZXAgYzI6IENP
TU1JVDsKIHN0ZXAgaW5zZXJ0MTogPC4uLiBjb21wbGV0ZWQ+Ci1zdGVwIHNlbGVjdDE6IFNFTEVD
VCAqIEZST00gdXBzZXJ0Oworc3RlcCBzZWxlY3QxOiBTRUxFQ1QgKiBGUk9NIHVwc2VydCBPUkRF
UiBCWSBpOwogaXwganwgIGsKIC0rLS0rLS0tCiAxfDExfDExMQpkaWZmIC0tZ2l0IGEvc3JjL3Rl
c3QvaXNvbGF0aW9uL3NwZWNzL2luc2VydC1jb25mbGljdC1kby11cGRhdGUtNC5zcGVjIGIvc3Jj
L3Rlc3QvaXNvbGF0aW9uL3NwZWNzL2luc2VydC1jb25mbGljdC1kby11cGRhdGUtNC5zcGVjCmlu
ZGV4IDYyOTdiMWQxZDZjLi5hNjI1MzE2NjBkMyAxMDA2NDQKLS0tIGEvc3JjL3Rlc3QvaXNvbGF0
aW9uL3NwZWNzL2luc2VydC1jb25mbGljdC1kby11cGRhdGUtNC5zcGVjCisrKyBiL3NyYy90ZXN0
L2lzb2xhdGlvbi9zcGVjcy9pbnNlcnQtY29uZmxpY3QtZG8tdXBkYXRlLTQuc3BlYwpAQCAtMjMs
NyArMjMsNyBAQCBzZXNzaW9uIHMxCiBzZXR1cCAgICAgICAgICAgeyBCRUdJTiBJU09MQVRJT04g
TEVWRUwgUkVBRCBDT01NSVRURUQ7IH0KIHN0ZXAgaW5zZXJ0MSAgICB7IElOU0VSVCBJTlRPIHVw
c2VydCBWQUxVRVMgKDEsIDExLCAxMTEpCiAgICAgICAgICAgICAgICAgICBPTiBDT05GTElDVCAo
aSkgRE8gVVBEQVRFIFNFVCBrID0gRVhDTFVERUQuazsgfQotc3RlcCBzZWxlY3QxICAgIHsgU0VM
RUNUICogRlJPTSB1cHNlcnQ7IH0KK3N0ZXAgc2VsZWN0MSAgICB7IFNFTEVDVCAqIEZST00gdXBz
ZXJ0IE9SREVSIEJZIGk7IH0KIHN0ZXAgYzEgICAgICAgICB7IENPTU1JVDsgfQogCiBzZXNzaW9u
IHMyCi0tIAoyLjM5LjIgKEFwcGxlIEdpdC0xNDMpCgo=
--00000000000036610a064b942a1c--