diff options
| author | Amit Langote <amitlan@postgresql.org> | 2024-11-08 16:30:33 +0900 |
|---|---|---|
| committer | Amit Langote <amitlan@postgresql.org> | 2024-11-08 17:18:38 +0900 |
| commit | 62df5484f976b76c95cbb00ae797e90de1b81f99 (patch) | |
| tree | 2945037d53a609dac3df04fa5c05a78580830a2a /src/backend/optimizer/util/relnode.c | |
| parent | 96f9b29a3e1ea3dd58e728814bf630d40ff77caa (diff) | |
| download | postgresql-62df5484f976b76c95cbb00ae797e90de1b81f99.tar.gz postgresql-62df5484f976b76c95cbb00ae797e90de1b81f99.zip | |
Disallow partitionwise join when collations don't match
If the collation of any join key column doesn’t match the collation of
the corresponding partition key, partitionwise joins can yield incorrect
results. For example, rows that would match under the join key collation
might be located in different partitions due to the partitioning
collation. In such cases, a partitionwise join would yield different
results from a non-partitionwise join, so disallow it in such cases.
Reported-by: Tender Wang <tndrwang@gmail.com>
Author: Jian He <jian.universality@gmail.com>
Reviewed-by: Tender Wang <tndrwang@gmail.com>
Reviewed-by: Junwang Zhao <zhjwpku@gmail.com>
Discussion: https://postgr.es/m/CAHewXNno_HKiQ6PqyLYfuqDtwp7KKHZiH1J7Pqyz0nr+PS2Dwg@mail.gmail.com
Backpatch-through: 12
Diffstat (limited to 'src/backend/optimizer/util/relnode.c')
| -rw-r--r-- | src/backend/optimizer/util/relnode.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/backend/optimizer/util/relnode.c b/src/backend/optimizer/util/relnode.c index 836bdaac069..a496f56b4d4 100644 --- a/src/backend/optimizer/util/relnode.c +++ b/src/backend/optimizer/util/relnode.c @@ -1807,6 +1807,10 @@ have_partkey_equi_join(RelOptInfo *joinrel, if (ipk1 != ipk2) continue; + /* Reject if the partition key collation differs from the clause's. */ + if (rel1->part_scheme->partcollation[ipk1] != opexpr->inputcollid) + return false; + /* * The clause allows partitionwise join only if it uses the same * operator family as that specified by the partition key. |