> On Jul 21, 2021, at 8:09 AM, Robert Haas <robertmh...@gmail.com> wrote:
>
> A callback where?
If you were going to support lots of formats, not just tar, you might want the
streamer class for each format to have a callback which sets up the injector,
rather than having CreateBackupStreamer do it directly. Even then, having now
studied CreateBackupStreamer a bit more, the idea seems less appealing than it
did initially. I don't think it makes things any cleaner when only supporting
tar, and maybe not even when supporting multiple formats, so I'll withdraw the
suggestion.
—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
