Fabien COELHO <coelho@cri.ensmp.fr> writes:
>> Masahiko Sawada <sawada.mshk@gmail.com> writes:
>>> [ pgbench_custom_initialization_v16.patch ]
>> I'm starting to review this patch, and I wonder how it is that you
>> ended up with "c" as the command letter for dropping existing tables.
>> Seems like "d" for DROP would be much less confusing. I see that at
>> one point "d" meant the data load step, but since you've gone with
>> "g" for "generate data" that conflict is gone.
> Indeed, you are right. As a reviewer, I can recall that there were some
> hesitations, not sure we ended up with the best possible choice.
OK, will make the appropriate changes.
> Note that if "c" is freed by "d" (drop), then it may be worth considering
> that "t" (table) could be replaced by "c" (create).
I thought about that, but the argument that 'c' might mean different
sorts of create steps (e.g. create index) seemed reasonable. I think
we're best off leaving it as 't' in case of future expansion.
regards, tom lane