Working through this ... what is the rationale for having changed
the API of logicalrep_read_update? It seems kind of random,
especially since no comparable change was made to
logicalrep_read_insert. If there's actually a good reason,
it seems like it'd apply to both. If there's not, I'd be
inclined to not change the API, because this sort of thing
is a recipe for bugs when making cross-version patches.
regards, tom lane