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:44 -0500 |
| commit | 5aa03f99fe7cfb1072849960465a040e834a25e2 (patch) | |
| tree | d1db89143fa443b90fdba72287f7a81fc08348e9 | |
| parent | 3037b28b89b44fccfcfddb2254655f45b3005f82 (diff) | |
| download | postgresql-5aa03f99fe7cfb1072849960465a040e834a25e2.tar.gz postgresql-5aa03f99fe7cfb1072849960465a040e834a25e2.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 d58237f5a15..d5a1017985c 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -5063,7 +5063,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: |