| From: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
|---|---|
| To: | Koshino Taiki <koshino(at)sraoss(dot)co(dot)jp> |
| Cc: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: doc: Reformat SELECT queries using GRAPH_TABLE |
| Date: | 2026-08-26 10:14:59 |
| Message-ID: | CAHGQGwHP4voXA5DU82hjqUTZarwSmsFeKG7FXSW+zCqaYgNCLA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Aug 26, 2026 at 10:25 AM Koshino Taiki <koshino(at)sraoss(dot)co(dot)jp> wrote:
> I wasn't sure whether this should be part of the same patch,
> so I've attached two versions: v3 leaves it untouched,
> and v4 includes the change. Let me know which you'd prefer.
I think v3 is preferable. The corresponding relational query in
ddl.sgml is also long, but it isn't a GRAPH_TABLE example, so I think
we should keep this patch focused on reformatting GRAPH_TABLE examples.
If necessary, you could start a separate thread about reformatting long
query examples in the documentation. It might be better to review all
such examples and reformat them consistently, rather than addressing
only the long query in ddl.sgml mentioned here.
-GRAPH_TABLE (mygraph MATCH (p IS person)-[h IS has]->(a IS account)
- COLUMNS (p.name AS person_name, h.since AS
has_account_since, a.num AS account_number)
+GRAPH_TABLE (mygraph
+ MATCH (p IS person)-[h IS has]->(a IS account)
+ COLUMNS (p.name AS person_name, h.since AS has_account_since,
a.num AS account_number)
Regarding v3 patch, this example in queries.sgml seems to be missing
the closing parenthesis for GRAPH_TABLE. That is, I think ")" should be
added at the end of the example above.
BTW, it would be helpful if you could trim the quoted text in your replies,
leaving only the parts relevant to your response. There's no need to
include the entire previous email at the bottom.
Regards,
--
Fujii Masao
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit Kapila | 2026-08-26 10:21:05 | Re: Assertion failure in GetSubscriptionRelations() with concurrent DROP TABLE |
| Previous Message | cca5507 | 2026-08-26 10:07:30 | Re: Walreceiver create temp slot more than once when timeline switch |