Hi Corey, On Wed, Jun 17, 2026 at 7:50 AM Corey Huinker <[email protected]> wrote: > I think the right first step is to create the functions > import_relation_stats() and import_attribute_stats() which take the oid of > the destination relation and the rest of the parameters are const char > pointers, which is the ideal case for a bunch of values being fetched from > PQgetValue(). Those functions will for now just make the call to > attribute_statistics_update() / relation_statistics_update(), but at least it > will be localized inside relation_stats.c and attribute_stats.c where we can > make internal refactors without disturbing anyone else.
+1 > Now, we *could* make these 2 new functions take the oid/oid+attname followed > by a variadic array of keyname+value string pairs like the existing > pg_restore_*_stats() functions, so the functions call signature would be > somewhat future-proofed. Let me know if that violates your vision of what > this should be. IMO I don't think we need to go that far, because 1) the new functions are provided for FDW authors only, and 2) we don't make changes to the stats that often. Thanks! Best regards, Etsuro Fujita
