Re: NULL handling in exconfig deconstruction

2018-11-12 Thread Tom Lane
Daniel Gustafsson writes: > In extension_config_remove() we first ensure that pg_extension.extconfig > cannot > contain any NULL values in the array, ERRORing out if so. Later we however > ask > for NULL values back when deconstructing the array, throwing away the results > knowing there wont b

NULL handling in exconfig deconstruction

2018-11-08 Thread Daniel Gustafsson
In extension_config_remove() we first ensure that pg_extension.extconfig cannot contain any NULL values in the array, ERRORing out if so. Later we however ask for NULL values back when deconstructing the array, throwing away the results knowing there wont be any, which seems superfluous (and wrong