PG docs - Sequence CYCLE clause
Hi hackers.
While reviewing another thread I had cause to be looking more
carefully at the SEQUENCE documentation.
I found it curious that, unlike other clauses, the syntax of the CYCLE
clause was not displayed on a line by itself.
I have changed that, and at the same time I have moved the CYCLE
syntax (plus its description) to be adjacent to MINVALUE/MAXVALUE,
which IMO is where it naturally belongs.
Please see the attached patch.
Thoughts?
======
Kind Regards,
Peter Smith.
Fujitsu Australia
Attachments:
docs_sequence_cycle.diffapplication/octet-stream; name=docs_sequence_cycle.diffDownload+62-59
On 13.08.24 07:46, Peter Smith wrote:
While reviewing another thread I had cause to be looking more
carefully at the SEQUENCE documentation.I found it curious that, unlike other clauses, the syntax of the CYCLE
clause was not displayed on a line by itself.I have changed that, and at the same time I have moved the CYCLE
syntax (plus its description) to be adjacent to MINVALUE/MAXVALUE,
which IMO is where it naturally belongs.Please see the attached patch.
I agree, it makes a bit more sense with your patches.
On Wed, Aug 14, 2024 at 01:46:23PM +0200, Peter Eisentraut wrote:
On 13.08.24 07:46, Peter Smith wrote:
While reviewing another thread I had cause to be looking more
carefully at the SEQUENCE documentation.I found it curious that, unlike other clauses, the syntax of the CYCLE
clause was not displayed on a line by itself.I have changed that, and at the same time I have moved the CYCLE
syntax (plus its description) to be adjacent to MINVALUE/MAXVALUE,
which IMO is where it naturally belongs.Please see the attached patch.
I agree, it makes a bit more sense with your patches.
Great, patch applied to master.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.