On Tue, May 9, 2017 at 4:26 AM, Dave Page <dp...@pgadmin.org> wrote:

>
> Anyhoo, The standard we try to use in pgAdmin at the moment is slightly
> different from the example you've given. To add it into the mix...
>
> /* Keywords Upper, 4 space indent, commas after, AND/OR after,
>  * no spaces after ( or before ), or around ::
>  */
> SELECT DISTINCT
>     dep.deptype,
>     dep.classid,
>     coalesce(coc.relname, clrw.relname) AS ownertable
> FROM
>     pg_depend dep
>     LEFT JOIN pg_class cl ON dep.objid = cl.oid
>     LEFT JOIN pg_attribute att ON dep.objid = att.attrelid AND
> dep.objsubid = att.attnum
> WHERE
>     dep.objid = 16385::oid AND
>     classid IN (
>         SELECT
>             oid
>         FROM
>             pg_class
>         WHERE
>             relname IN ('pg_class', 'pg_constraint')
>     )
> ORDER BY
>     classid,
>     cl.relkind;
>
> There are some subtleties that are open to personal taste there;
>
> - Formatting of the sub-select - e.g. should the SELECT directly follow
> the (, and should the rest be indented accordingly?
>
> - Formatting of multiple quals in the joins; e.g. should the qual
> following the AND be on the next line, and if so, should it be indented one
> level, or to align with the qual above? Should the first qual be on the
> next line?
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Sorry I let this linger. Let's go with the style you currently have. With
respect to your additional comments.

- Sub-Select should be a new line and indented as in your example.
- For multiple quals in the joins; the qual following the AND should be on
the next line, and should it be indented one level

-- Rob

Reply via email to