From: | Dave Page <dpage(at)postgresql(dot)org> |
---|---|
To: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: feature request: expose sql formatter |
Date: | 2007-02-06 20:39:44 |
Message-ID: | 45C8E790.8090306@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Merlin Moncure wrote:
> hello all, I have a pie in the sky feature request here. I've been
> looking around for a sql 'beautifier' so that all our ddl which is
> stored in .sql files can be cleaned up. pgadmin has a much better
> formatter than the gook emitted by pg_dump...
>
> couple of questions here:
> * would it be possible to call pgadmin with a switch from the command
> line so it formats an sql file and exits?
>
> pgadmin -f foo.sql > foo_formatted.sql
>
> * the default options are pretty good, but what about improving the
> flexibility a bit (for example, sql keywords upper/lower case)? this
> would benefit everybody imo.
>
> if you guys are open this, could somebody give some hints about the
> complexity?
Hi Merlin,
The architecture of pgAdmin is such that your first suggestion would be
nigh-on impossible. Each object type has a class which is able to read
one or more objects of it's own type from the catalog, and provide a set
of members to access it's properties. There are additional member
functions to return the SQL, but this is reconstructed from the
properties of the object, not by re-formatting the output of
pg_get_viewdef or it's relatives.
The second suggestion is doable, but fairly invasive as you'd need to
get a range of conditional statements to format different parts of the
DDL created in each GetSQL() member. I'd be wary of accepting a patch to
do this due to the mess it would likely make of much of the code.
Regards, Dave.
From | Date | Subject | |
---|---|---|---|
Next Message | Merlin Moncure | 2007-02-07 13:47:09 | Re: feature request: expose sql formatter |
Previous Message | Merlin Moncure | 2007-02-06 16:10:25 | feature request: expose sql formatter |