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:24 +0000 (15:11 -0500)
commit	8b90174e3f0638ff542bef7f367429750d18b08c
tree	774da5e7909a5552208dc0b2f5043c0a61746928	tree \| snapshot
parent	a0696d29551bf3cb5dbd2b72f89aed0bbf0dcab3	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