On Fri, Feb 25, 2022 at 7:08 AM Japin Li <japi...@hotmail.com> wrote:
> > On Fri, 25 Feb 2022 at 20:48, David Christensen < > david.christen...@crunchydata.com> wrote: > >> Cool. I think we can report an error instead of reading wal files, > >> if the tablespace, database, or relation is invalid. Does there any > >> WAL record that has invalid tablespace, database, or relation OID? > > > > The only sort of validity check we could do here is range checking for > the underlying data types > > (which we certainly could/should add if it’s known to never be valid for > the underlying types); > > The invalid OID I said here is such as negative number and zero, for those > parameters, we do not need to read the WAL files, since it always invalid. > Agreed. Can add some additional range validation to the parsed values. David