Skip to content

Commit 4e7c307

Browse files
authored
Refactor make_range into compat function (#8327)
Adds `make_range_compat` to make the code cleaner, since we now use it in a few places. Disable-check: force-changelog-file
1 parent 46b05f2 commit 4e7c307

3 files changed

Lines changed: 11 additions & 15 deletions

File tree

src/compat/compat.h

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -691,3 +691,11 @@ pg_cmp_u32(uint32 a, uint32 b)
691691
norderbys) \
692692
index_beginscan(heapRelation, indexRelation, snapshot, instrument, nkeys, norderbys)
693693
#endif
694+
695+
#if PG16_LT
696+
#define make_range_compat(typcache, lower, upper, empty, escontext) \
697+
make_range(typcache, lower, upper, empty)
698+
#else
699+
#define make_range_compat(typcache, lower, upper, empty, escontext) \
700+
make_range(typcache, lower, upper, empty, escontext)
701+
#endif

src/time_utils.c

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -615,11 +615,7 @@ ts_make_range_from_internal_time(PG_FUNCTION_ARGS)
615615

616616
/* Need to check the types of the lower and upper values. They should
617617
* match the returned range. */
618-
#if PG16_LT
619-
PG_RETURN_RANGE_P(make_range(typcache, &lower, &upper, false));
620-
#else
621-
PG_RETURN_RANGE_P(make_range(typcache, &lower, &upper, false, escontext));
622-
#endif
618+
PG_RETURN_RANGE_P(make_range_compat(typcache, &lower, &upper, false, escontext));
623619
}
624620

625621
Datum

tsl/src/bgw_policy/continuous_aggregate_api.c

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -559,11 +559,7 @@ policy_refresh_cagg_check_for_overlaps(ContinuousAgg *cagg, Jsonb *policy_config
559559
if (typcache == NULL || typcache->rngelemtype == NULL)
560560
elog(ERROR, "cache lookup failed");
561561

562-
#if PG16_LT
563-
RangeType *range = make_range(typcache, &lower, &upper, false);
564-
#else
565-
RangeType *range = make_range(typcache, &lower, &upper, false, NULL);
566-
#endif
562+
RangeType *range = make_range_compat(typcache, &lower, &upper, false, NULL);
567563

568564
ListCell *lc;
569565

@@ -606,11 +602,7 @@ policy_refresh_cagg_check_for_overlaps(ContinuousAgg *cagg, Jsonb *policy_config
606602
.lower = false,
607603
};
608604

609-
#if PG16_LT
610-
RangeType *range_job = make_range(typcache, &lower_job, &upper_job, false);
611-
#else
612-
RangeType *range_job = make_range(typcache, &lower_job, &upper_job, false, NULL);
613-
#endif
605+
RangeType *range_job = make_range_compat(typcache, &lower_job, &upper_job, false, NULL);
614606

615607
elog(DEBUG1,
616608
"start_offset_job: " INT64_FORMAT ", end_offset_job: " INT64_FORMAT,

0 commit comments

Comments
 (0)