> On Mar 11, 2021, at 12:12 AM, Peter Eisentraut 
> <peter.eisentr...@enterprisedb.com> wrote:
> 
> (I'm still not a fan of adding more client-side tools whose sole task is to 
> execute server-side functionality in a slightly filtered way, but it seems 
> people are really interested in this, so ...)
> 
> I want to register, if we are going to add this, it ought to be in src/bin/.  
> If we think it's a useful tool, it should be there with all the other useful 
> tools.

I considered putting it in src/bin/scripts where reindexdb and vacuumdb also 
live.  It seems most similar to those two tools.

> I realize there is a dependency on a module in contrib, and it's probably now 
> not the time to re-debate reorganizing contrib.  But if we ever get to that, 
> this program should be the prime example why the current organization is 
> problematic, and we should be prepared to make the necessary moves then.

Before settling on contrib/pg_amcheck as the location, I checked whether any 
tools under src/bin had dependencies on a contrib module, and couldn't find any 
current examples.  (There seems to have been one in the past, though I forget 
which that was at the moment.)

I have no argument with changing the location of this tool before it gets 
committed, but I wonder if we should do that now, or wait until some future 
time when contrib gets reorganized?  I can't quite tell which you prefer from 
your comments above.

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company





Reply via email to