Patch looks good, all concerns that had been raised previously have
been addressed in this version of the patch.
The only thing that IMO needs to change is a stylistic issue:
if (fout->remoteVersion >= 80200)
{ [...] (fout->remoteVersion >= 80400) ?
"pg_catalog.pg_get_function_identity_arguments(oid)" : "NULL::text", [...]
}
Please just create a whole new
if (fout->remoteVersion >= 80400)
{ [...]
}
here.
Other than that, the feature works as advertised and does not
negatively affect runtime or memory consumption (the new field is only
added to functions / aggregates).
Please send a new version of the patch that changes the above
mentioned item, the patch also needs rebasing (off by a couple of
lines).
