diff options
| author | Robert Haas | 2017-05-17 18:31:48 +0000 |
|---|---|---|
| committer | Robert Haas | 2017-05-17 18:31:48 +0000 |
| commit | b2e4399baaa805a046b7ffa155c666c80bbf8429 (patch) | |
| tree | a0d5a4b0be4f3bda543181c21ec16135d2d4c4df | |
| parent | 05b5feb60e7cf36db08eb0d34dbd21854cb9e8ce (diff) | |
Code review for make_partition_op_expr.
It's better to use the actual keynum here rather than 0, because
someday someone might try to make list partitioning work with
multiple partitioning columns.
Jeevan Ladhe
Discussion: http://postgr.es/m/CAOgcT0M6-mx+dSX47JGJuJP1CKr4XssBFVmKNETt0OZYWpFr+w@mail.gmail.com
| -rw-r--r-- | src/backend/catalog/partition.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/catalog/partition.c b/src/backend/catalog/partition.c index 2e9b727fe78..acd9f2d273e 100644 --- a/src/backend/catalog/partition.c +++ b/src/backend/catalog/partition.c @@ -1266,7 +1266,7 @@ make_partition_op_expr(PartitionKey key, int keynum, saopexpr->opno = operoid; saopexpr->opfuncid = get_opcode(operoid); saopexpr->useOr = true; - saopexpr->inputcollid = key->partcollation[0]; + saopexpr->inputcollid = key->partcollation[keynum]; saopexpr->args = list_make2(arg1, arg2); saopexpr->location = -1; |