Re: Minor code improvement to estimate_path_cost_size in postgres_fdw

2018-01-11 Thread Tom Lane
Tatsuro Yamada writes: > The declaration of estimate_path_cost_size uses baserel, but > the actual definition uses foreignrel. It would be better to sync. Yeah, the join_conds parameter's been renamed at some point too :-( Fixed. regards,

Minor code improvement to estimate_path_cost_size in postgres_fdw

2018-01-11 Thread Tatsuro Yamada
Hi, The declaration of estimate_path_cost_size uses baserel, but the actual definition uses foreignrel. It would be better to sync. Please find attached a patch. Tatsuro Yamada NTT Open Source Software Center diff --git a/contrib/postgres_fdw/postgres_fdw.c