Re: Patch to add 'id'

From: "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Re: Patch to add 'id'
Date: 2018-06-10 21:06:30
Message-ID: 863CC1B5-D159-42FC-8FA5-830C7BD855CF@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www


> On Jun 10, 2018, at 3:29 PM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
>> On 10 Jun 2018, at 04:16, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>>
>> Would someone please apply this patch to the website code so pglife
>> (http://pglife.momjian.us/) can more easily scrape the current release
>> numbers from the website? Thanks.
>
> While harmless enough, I’m not a huge fan of adding identifiers to the site
> which are not in use in the site code, such non-obvious external dependencies
> make the code harder to maintain.

Yes, while I’d love to support the version feed into pglife, I’m a -1 in
general on this patch:

1. There was a glut of IDs in the old styles and I don’t want to
start adding more unless there is a specific reason necessary for
supporting functionality on pgweb itself.

2. Typically websites make “structured information” as such available
via APIs or other interfaces, which I believe Magnus points out in a different
reply is achievable via a RSS feed[1].

I don’t think we should be adding more classes/IDs just for structured
information in the markup unless it’s being used to support pgweb
itself, because as Daniel points out it does become a maintenance issue
(speaking from firsthand experience).

If we do want to have more “structured” resources, I’d suggest either more
feeds (which seems to be the current PostgreSQL way of making things
available). The reason I’m not suggesting as RESTful API is I don’t see
the need for one just yet, as that has its own maintainability issues.

Thanks,

Jonathan

[1] https://www.postgresql.org/versions.rss <https://www.postgresql.org/versions.rss> - appears to be working too! :-)

In response to

Browse pgsql-www by date

  From Date Subject
Next Message Jonathan S. Katz 2018-06-10 21:46:05 Re: Patch to add 'id'
Previous Message Magnus Hagander 2018-06-10 19:40:20 Re: Patch to add 'id'