Skip to content

Commit cce58a8

Browse files
authored
Fix missing GUC_GPDB_NEED_SYNC for pax filter GUCs (#1590)
Add GUC_GPDB_NEED_SYNC flag to pax.enable_sparse_filter and pax.enable_row_filter so their values are dispatched from QD to QE segments. Without this flag, SET on the coordinator has no effect because scans run on QE segments.
1 parent 62a13fb commit cce58a8

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • contrib/pax_storage/src/cpp/comm

contrib/pax_storage/src/cpp/comm/guc.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,11 +134,11 @@ void DefineGUCs() {
134134
"enable pax filter, contains min/max and bloom "
135135
"filters for sparse filtering",
136136
NULL, &pax::pax_enable_sparse_filter, true,
137-
PGC_USERSET, 0, NULL, NULL, NULL);
137+
PGC_USERSET, GUC_GPDB_NEED_SYNC, NULL, NULL, NULL);
138138

139139
DefineCustomBoolVariable("pax.enable_row_filter", "enable pax row filter",
140140
NULL, &pax::pax_enable_row_filter, false,
141-
PGC_USERSET, 0, NULL, NULL, NULL);
141+
PGC_USERSET, GUC_GPDB_NEED_SYNC, NULL, NULL, NULL);
142142

143143
DefineCustomIntVariable(
144144
"pax.scan_reuse_buffer_size", "set the reuse buffer size", NULL,

0 commit comments

Comments
 (0)