Hi Justin,
Thanks for your comments!
>> Currently, DBAs need the table name to see the constraint information.
>
> Or, they can query pg_constraint or information_schema: check_constraints,
> table_constraints.
Yeah, right.
If they can use the meta-command instead of a long query against pg_constraint
or information_schema and also pg_attribulte, it would be helpful, I believe. :-D
>> - success = listConversions(pattern, show_verbose, show_system);
>> + if (strncmp(cmd, "dco", 3) == 0) /* Constraint */
>> + switch (cmd[3])
>> + {
>> + case '\0':
>> + case '+':
>
> Does "+" do anything ?
No, it doesn't. Removed.
>> +++ b/src/bin/psql/help.c
>> @@ -231,6 +231,7 @@ slashUsage(unsigned short int pager)
>
>> fprintf(output, _(" \\db[+] [PATTERN] list tablespaces\n"));
>> fprintf(output, _(" \\dc[S+] [PATTERN] list conversions\n"));
>> + fprintf(output, _(" \\dco[S] [PATTERN] list constraint\n"));
>
> Should be plural "constraints".
>
> I think "exclude" should be called "exclusion" ("exclude" sounded to me like
> you're going to provide a way to "exclude" types of constraints, like "xc"
> would show everything except check constraints).
Thanks! Fixed the both.
Attached file is new patch. It includes:
- Fix help message s/constraint/constraints/
- s/Exclude/Exclusion/
- Remove unused modifier "+"
- Add document for \dco
Thanks,
Tatsuro Yamada