Обсуждение: Making CopyFromStateData not internal anymore

Поиск
Список
Период
Сортировка

Making CopyFromStateData not internal anymore

От
Jelte Fennema
Дата:
For Citus we'd like to hook into the way that GENERATED AS IDENTITY
generates the next value. The way we had in mind was to replace
T_NextValueExpr with a function call node. But doing that same for
COPY seems only possible by manually changing the defexprs array of
the CopyFromState. Sadly CopyFromStateData is in an internal header so
that seems dangerous to do, since the struct definition might change
across minor versions.
However, it seems that the only time this was actually done in the
last 5 years was in 8dc49a8934de023c08890035d96916994bd9b297

What do you think about making CopyFromStateData its definition public?