Re: pgbench - allow to create partitioned tables

Hello Amit,

>> Yes, on -i it will fail because the syntax will not be recognized.
>
> Maybe we should be checking the server version, which would allow to
> produce more useful error messages when these options are used against
> older servers, like
>
> if (sversion < 10000)
>    fprintf(stderr, "cannot use --partitions/--partitions-method
> against servers older than 10");
>
> We would also have to check that partition-method=hash is not used against v10.
>
> Maybe overkill?

Yes, I think so: the error detection and messages would be more or less 
replicated from the server and would vary from version to version.

I do not think that it is worth going this path because the use case is 
virtually void as people in 99.9% of cases would use a pgbench matching 
the server version. For those who do not, the error message should be 
clear enough to let them guess what the issue is. Also, it would be 
untestable.

One thing we could eventually do is just to check pgbench version against 
the server version like psql does and output a generic warning if they 
differ, but franckly I do not think it is worth the effort: ISTM that 
nobody ever complained about such issues. Also, that would be matter for 
another patch.

-- 
Fabien.