diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-01-21 18:33:32 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-01-21 18:33:43 -0500 |
| commit | 7fbdefd30e7c644bd7c6ec0799abb49584e01477 (patch) | |
| tree | d5d8fc720a02f21c17c79d86be33b00a38c660fc | |
| parent | 123cc697a8eb0827e82ceea3f6da55b2f05eb422 (diff) | |
| download | postgresql-7fbdefd30e7c644bd7c6ec0799abb49584e01477.tar.gz postgresql-7fbdefd30e7c644bd7c6ec0799abb49584e01477.zip | |
Remove useless bms_copy step in RelationGetIndexAttrBitmap.
Seems to be from a bad case of copy-and-paste-itis in commit 665d1fad9.
It wouldn't be quite so annoying if it didn't contradict the comment
half a dozen lines above.
David Rowley
Discussion: https://postgr.es/m/CAKJS1f95Dyf8Qkdz4W+PbCmT-HTb54tkqUCC8isa2RVgSJ_pXQ@mail.gmail.com
| -rw-r--r-- | src/backend/utils/cache/relcache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c index e21ec620db1..84becb9f3fb 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -5003,7 +5003,7 @@ restart: case INDEX_ATTR_BITMAP_KEY: return uindexattrs; case INDEX_ATTR_BITMAP_PRIMARY_KEY: - return bms_copy(relation->rd_pkattr); + return pkindexattrs; case INDEX_ATTR_BITMAP_IDENTITY_KEY: return idindexattrs; default: |