On Wed, Feb 4, 2026 at 8:53 AM Amul Sul <[email protected]> wrote: > > 0008. I'm concerned about this patch. It is breaking backward compatibility > > if > > you are using a long option (--wal-directory). Your proposal is a generic > > word > > that represents both cases (file and directory). I agree. However, I > > wouldn't > > remove --wal-directory from the tool. Instead, I would keep it with the same > > short option ('w') but add a sentence saying this long option is deprecated > > and > > will be removed in the future or even remove any traces of this long option > > from the help and documentation but silently accept the old long option. I > > prefer the latter because it is not a required argument so a deprecation > > warning is not necessary IMO. > > Yeah, that was discussed with Robert offline and we believe that it is > better to make it more generalized; since we can now use the same > option to accept both wal-directory and wal-archived. pg_waldump has > much more generic options for the same, such as -- path=PATH.
Of course, the fact that we discussed it doesn't mean that the issue is completely settled. However, I don't think there would be general support from other people on the project for the idea of getting rid of the long option entirely, or even just the documentation for it. We have long options for almost all short options these days, and I agree with that as a general practice. Sometimes we have ONLY a long option, but we very rarely have ONLY a short option, which is good because sometimes we have tools with too many options for it to be viable to give everything a short option, and the long options really help to make things more self-documenting. What I would consider a more viable option is to not do the rename and leave this as --wal-directory even though the argument could really be a directory or a file. That would avoid the backward compatibility break that is troubling Euler. And, you could argue that a tar file is enough like a directory that it won't really cause much confusion. Personally, I favor renaming it. I think the number of people using pg_waldump is fairly small, and the number of those people who are using the long form of the option is very small. Hence, I don't believe think the rename will inconvenience many users, and I think it will improve clarity for future users. But, Euler (or someone else) might take the opposite viewpoint. -- Robert Haas EDB: http://www.enterprisedb.com
