diff --git a/src/backend/columnar/columnar_metadata.c b/src/backend/columnar/columnar_metadata.c index f4549849a..cd62c8b0c 100644 --- a/src/backend/columnar/columnar_metadata.c +++ b/src/backend/columnar/columnar_metadata.c @@ -1323,9 +1323,9 @@ GetHighestUsedAddress(Relation rel) * In case if relid hasn't been defined yet, we should use RelidByRelfilenumber * to get correct relid value. * - * Now it is basically used for temp rels, because since PG13 RelidByRelfilenumber - * skip temp relations and we should use alternative ways to get relid value in case - * of temp objects. + * Now it is basically used for temp rels, because since PG18(it was backpatched + * through PG13) RelidByRelfilenumber skip temp relations and we should use + * alternative ways to get relid value in case of temp objects. */ Oid ColumnarRelationId(Oid relid, RelFileLocator relfilelocator) diff --git a/src/backend/columnar/columnar_writer.c b/src/backend/columnar/columnar_writer.c index 4a412508d..e698d1a41 100644 --- a/src/backend/columnar/columnar_writer.c +++ b/src/backend/columnar/columnar_writer.c @@ -47,7 +47,12 @@ struct ColumnarWriteState TupleDesc tupleDescriptor; FmgrInfo **comparisonFunctionArray; RelFileLocator relfilelocator; - Oid temp_relid; /* We can't rely on RelidByRelfilenumber for temp tables since PG18.*/ + + /* + * We can't rely on RelidByRelfilenumber for temp tables since + * PG18(it was backpatched through PG13). + */ + Oid temp_relid; MemoryContext stripeWriteContext; MemoryContext perTupleContext; diff --git a/src/include/columnar/columnar.h b/src/include/columnar/columnar.h index 1d6e462bd..1883be38b 100644 --- a/src/include/columnar/columnar.h +++ b/src/include/columnar/columnar.h @@ -65,9 +65,9 @@ #define CITUS_COLUMNAR_INTERNAL_VERSION "11.1-0" /* - * We can't rely on RelidByRelfilenumber for temp tables since PG13, so we can use - * this macro to define relid within relation in case of temp relations. Otherwise - * RelidByRelfilenumber should be used. + * We can't rely on RelidByRelfilenumber for temp tables since PG18(it was backpatched + * through PG13), so we can use this macro to define relid within relation in case of + * temp relations. Otherwise RelidByRelfilenumber should be used. */ #define RelationPrecomputeOid(a) (RelationUsesLocalBuffers(a) ? RelationGetRelid(a) : \ InvalidOid)