On Mon, Oct 12, 2020 at 7:32 PM Tomas Vondra <tomas.von...@2ndquadrant.com> wrote: > > On Mon, Oct 12, 2020 at 02:28:43PM +0530, Dilip Kumar wrote: > > > >> ... > > > >I have worked on this patch, so as discussed now I am maintaining the > >preserved compression methods using dependency. Still PRESERVE ALL > >syntax is not supported, I will work on that part. > > > > Cool, I'll take a look. What's your opinion on doing it this way? Do you > think it's cleaner / more elegant, or is it something contrary to what > the dependencies are meant to do?
I think this looks much cleaner. Moreover, I feel that once we start supporting the custom compression methods then we anyway have to maintain the dependency so using that for finding the preserved compression method is good option. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com