On Tue, 2022-11-29 at 16:26 -0500, Kirk Wolak wrote: > On Mon, Nov 28, 2022 at 12:45 AM Tom Lane <tgl@sss.pgh.pa.us> wrote: > > Laurenz Albe <laurenz.albe@cybertec.at> writes: > > > Now I think that is taking it too far. Your code sample would be great > > > for a tutorial, but it is too elaborate for the technical documentation. > > > The example should focus on the sequence functions, but more than half > > > of the code describes other parts of PostgreSQL: > > > > Yeah, that stuff seems quite out of place here. > > > > > I am alright with having a CREATE TABLE with a DEFAULT and an INSERT or two; > > > whatever it takes to show the functions in a realistic scenario. > > > For example, you could INSERT a row that overrides the DEFAULT, then call > > > "setval()" to readjust the sequence. > > > > I don't believe we have such detail around very many, if indeed any, > > of our other functions' documentation. > > > > regards, tom lane > > > All changes specified have been addressed. > The Example is significantly reduced, but readable. > The extra "SELECT "'s have been removed off of the inline examples, excluding the existing paragraph. > > This is the smallest possible change, that still has an example. > > The "Example" is not <title> as every attempt to make it such fails the LINT process.
You can have <title> only after the start of a section.
The new examples in the table don't really add anything to the function declaration...
I realized that there already are some examples in the CREATE SEQUENCE documentation, so what about linking there instead of writing more examples?
The attached patch does that and removes the less useful examples. What do you think?