Re: [PATCH] Psql List Languages

Em Thu, 02 Jul 2009 15:27:33 -0400
Tom Lane <tgl@sss.pgh.pa.us> escreveu:

> Peter Eisentraut <peter_e@gmx.net> writes:
> > Looks good, but needs documentation and tab-complete.c updates, it
> > seems.  And you should revisit this patch to make it consistent
> > with the S flag that was added to most \d commands.  For example,
> > \dL would show only user-added languages, but \dLS would include c,
> > internal, and sql.
>
> Another recent change in the \d support is that commands are expected
> to work (as best they can) for all server versions back to 7.4.  I
> didn't look to see if this patch needs any fixes for that, but it's
> something to keep in mind while reviewing.



  I applied the Tom Lane and Peter considerations, but I had that
remove one column (Owner) of out command \dL to compatibility with 7.4
version. The command \dLS output:

test=# \dLS
                                      List of languages
   Name   | Procedural Language |  Trusted  |     Call Handler     |
        Validator
----------+---------------------+-----------+----------------------+
-------------------------
 c        | Untrusted           | Untrusted |                      |
 fmgr_c_validator
 internal | Untrusted           | Untrusted |                      |
 fmgr_internal_validator
 plperl   | Trusted             | Trusted   | plperl_call_handler  |
 plperl_validator
 plpgsql  | Trusted             | Trusted   | plpgsql_call_handler |
 plpgsql_validator
 pltcl    | Trusted             | Trusted   | pltcl_call_handler   |

 sql      | Untrusted           | Trusted   |                      |
 fmgr_sql_validator
(6 rows)

test=#


  Comments?



Best Regards,
--
Fernando Ike