granicus.if.org Git - postgresql/commit

]> granicus.if.org Git - postgresql/commit

author	Tom Lane <tgl@sss.pgh.pa.us>
	Thu, 13 Dec 2018 20:11:09 +0000 (15:11 -0500)
committer	Tom Lane <tgl@sss.pgh.pa.us>
	Thu, 13 Dec 2018 20:11:16 +0000 (15:11 -0500)
commit	171cae1579124a400e234f89c1fc80f3caed5f68
tree	f693c5e6dafb707bb3a882bc3ba3f91a270cf460	tree \| snapshot
parent	dd951dc34a2ecde28cd8d686cd06de1d21f474a0	commit \| diff

Fix bogus logic for skipping unnecessary partcollation dependencies.

The idea here is to not call recordDependencyOn for the default collation,
since we know that's pinned. But what the code actually did was to record
the partition key's dependency on the opclass twice, instead.

Evidently introduced by sloppy coding in commit 2186b608b. Back-patch
to v10 where that came in.

src/backend/catalog/heap.c

diff | blob | history

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom