On Mon, Sep 26, 2016 at 6:15 PM, Ashutosh Bapat < ashutosh.ba...@enterprisedb.com> wrote:
> This patch will need some changes to conversion_error_callback(). That > function reports an error in case there was an error converting the > result obtained from the foreign server into an internal datum e.g. > when the string returned by the foreign server is not acceptable by > local input function for the expected datatype. In such cases, the > input function will throw error and conversion_error_callback() will > provide appropriate context for that error. postgres_fdw.sql has tests > to test proper context > We need to fix the error context to provide meaningful information or > at least not crash. This has been discussed briefly in [1]. > Oops. I had that in mind when working on this. Somehow skipped checking for conversion error context. I have fixed that in v3 patch. Removed assert and for non Var expressions, printing generic context. This context is almost in-line with the discussion you referred here. Thanks -- Jeevan B Chalke Principal Software Engineer, Product Development EnterpriseDB Corporation The Enterprise PostgreSQL Company