On Tue, Oct 5, 2021 at 5:32 AM Bossart, Nathan <bossa...@amazon.com> wrote:
> On 10/4/21, 8:19 PM, "Stephen Frost" <sfr...@snowman.net> wrote: > > It's also been discussed, at least around the water cooler (as it were > > in pandemic times- aka our internal slack channels..) that the existing > > archive command might be reimplemented as an extension using these. Not > > sure if that's really necessary but it was a thought. In any case, > > thanks for working on this! > > Interesting. I like the idea of having one code path for everything > instead of branching for the hook and non-hook paths. Thanks for > sharing your thoughts. > I remember having had this discussion a few times, I think mainly with Stephen and David as well (but not on their internal slack channels :P). I definitely think that's the way to go. It gives a single path for everything which makes it simpler in the most critical parts. And once you have picked an implementation other than it, you're now completely rid of the old implementation. And of course the good old idea that having an extension already using the API is a good way to show that the API is in a good place. As much as I dislike our current interface in archive_command, and would like to see it go away completely, I do believe we need to ship something that has it - if nothing else then for backwards compatibility. But an extension like this would also make it easier to eventually, down the road, deprecate this solution. Oh, and please put said implementation in a better place than contrib :) -- Magnus Hagander Me: https://www.hagander.net/ <http://www.hagander.net/> Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>
