Re: Fix typos and inconsistencies for HEAD (take 11)

2019-08-21 Thread Michael Paquier
On Tue, Aug 20, 2019 at 04:47:41PM -0400, Alvaro Herrera wrote: > Yeah, Alexander proposed change is correct. I just pushed it. Thanks, Alvaro. -- Michael signature.asc Description: PGP signature

Re: Fix typos and inconsistencies for HEAD (take 11)

2019-08-20 Thread Alvaro Herrera
On 2019-Aug-19, Michael Paquier wrote: > On Mon, Aug 19, 2019 at 07:04:04AM +0300, Alexander Lakhin wrote: > > 11.23 TupleLockUpdate -> LockTupleNoKeyExclusive > > Not sure about this one, so discarded for now. Alvaro? Yeah, Alexander proposed change is correct. I just pushed it. > > 11.33 vi

Re: Fix typos and inconsistencies for HEAD (take 11)

2019-08-19 Thread Michael Paquier
On Mon, Aug 19, 2019 at 04:22:44PM +0900, Michael Paquier wrote: > On Mon, Aug 19, 2019 at 07:04:04AM +0300, Alexander Lakhin wrote: >> 11.57 PASSBYVALUE -> PASSEDBYVALUE > > Will fix this one separately and back-patch. No need to, actually, as the error comes from 7bdc655. -- Michael signature

Re: Fix typos and inconsistencies for HEAD (take 11)

2019-08-19 Thread Michael Paquier
On Mon, Aug 19, 2019 at 07:04:04AM +0300, Alexander Lakhin wrote: > 11.23 TupleLockUpdate -> LockTupleNoKeyExclusive Not sure about this one, so discarded for now. Alvaro? > 11.25 typstore -> tupstore This one is cute. It actually does not cause a compilation failure as tuplestore_donestoring(

Re: Fix typos and inconsistencies for HEAD (take 10)

2019-08-12 Thread Michael Paquier
On Sun, Aug 11, 2019 at 11:00:00AM +0300, Alexander Lakhin wrote: > 10.44. serendipitiously -> serendipitously I didn't know that this even was a word: https://www.thefreedictionary.com/serendipitously But it seems to come from Horace Walpole. > 10.50. sigsetmask -> pgsigsetmask It should be pqs

Re: Fix typos and inconsistencies for HEAD (take 10)

2019-08-11 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next heap of typos and inconsistencies in the tree: 10.1. query_txt -> query text 10.2. quote_object_names -> quote_object_name 10.3. ragetypes_typanalyze.c -> rangetypes_typanalyze.c 10.4. RAISE_EXCEPTION -> ERRCODE_RAISE_EXCEPTION 10.5. rb_lower -> remo

Re: Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Alexander Lakhin
05.08.2019 8:40, Michael Paquier wrote: > On Mon, Aug 05, 2019 at 06:44:46AM +0300, Alexander Lakhin wrote: >> I believe that all the fixes in doc/ should be back-patched too. If it's >> not too late, I can produce such patches for the nearest releases. > I think that's unfortunately a bit too late

Re: Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Michael Paquier
On Mon, Aug 05, 2019 at 06:44:46AM +0300, Alexander Lakhin wrote: > I believe that all the fixes in doc/ should be back-patched too. If it's > not too late, I can produce such patches for the nearest releases. I think that's unfortunately a bit too late for this release. Those things have been ar

Re: Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Michael Paquier
On Mon, Aug 05, 2019 at 12:15:21PM +0900, Michael Paquier wrote: > On Mon, Aug 05, 2019 at 12:33:34AM +0300, Alexander Lakhin wrote: >> 9.41. OWNER_TO -> OWNER TO > > This one needs a back-patch. I'll fix that separately. Done separately as of 05ba837, and back-patched down to 9.6 as this has be

Re: Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Alexander Lakhin
Hello Michael, 05.08.2019 6:15, Michael Paquier wrote: >> 9.41. OWNER_TO -> OWNER TO > This one needs a back-patch. I'll fix that separately. > I believe that all the fixes in doc/ should be back-patched too. If it's not too late, I can produce such patches for the nearest releases. >> 9.75. Poin

Re: Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Michael Paquier
On Mon, Aug 05, 2019 at 12:33:34AM +0300, Alexander Lakhin wrote: > 9.1. NAMESPACE_SQLXML -> remove (not used since the introduction in > 355e05ab) Looks important to keep as a matter of documentation. > 9.12. not_point -> is_point (a contradiction with the check and a > comment, appeared in 4c13

Re: Fix typos and inconsistencies for HEAD (take 8)

2019-07-28 Thread Michael Paquier
On Sun, Jul 28, 2019 at 07:44:44AM +0300, Alexander Lakhin wrote: > 8.3. lag_with_offset_and_default, * -> > window_lag_with_offset_and_default, window_* (in windowfuncs.c) The intention here is to refer to the SQL-visible names. > In passing, I found a legacy script, FAQ2txt, that should be dele

Re: Fix typos and inconsistencies for HEAD (take 7)

2019-07-21 Thread Alexander Lakhin
Hello Tom, 22.07.2019 7:14, Tom Lane wrote: >> Fixing both places sounds adapted to me. An alternative we could use >> here is just to say something like that: >> The effective resolution is only 1/HZ, which can be configured with >> kernel parameter (see man 7 time), and is 4 milliseconds by >> d

Re: Fix typos and inconsistencies for HEAD (take 7)

2019-07-21 Thread Alexander Lakhin
Hello Michael, 22.07.2019 4:05, Michael Paquier wrote: >> Also, I found e-mail headers in optimizer/plan/README not relevant, so I >> propose to remove them. > Not sure about that part. I agree that the proposed fix is not complete, but just raises the demand for a subsequent fix. If you don't mind

Re: Fix typos and inconsistencies for HEAD (take 7)

2019-07-21 Thread Tom Lane
Michael Paquier writes: > On Sun, Jul 21, 2019 at 08:28:53AM +0300, Alexander Lakhin wrote: >> And another finding is related to the sleep effective resolution. `man 7 >> time` says "Since kernel 2.6.13, the HZ value is a kernel configuration  >> parameter  and  can  be 100, 250 (the default) ..."

Re: Fix typos and inconsistencies for HEAD (take 7)

2019-07-21 Thread Michael Paquier
On Sun, Jul 21, 2019 at 08:28:53AM +0300, Alexander Lakhin wrote: > Please consider fixing the next pack of typos and inconsistencies in the > tree: Thanks, all those things look fine. I have noticed one mistake. > 7.44. json_plperl -> jsonb_plperlu The path was incorrect here. > Also, I found

Re: Fix typos and inconsistencies for HEAD (take 7)

2019-07-21 Thread Tom Lane
Alexander Lakhin writes: > Also, I found e-mail headers in optimizer/plan/README not relevant, so I > propose to remove them. FWIW, I think they're highly relevant, because they put a date on that text. I've not gone through that README lately, but I wouldn't be surprised if it's largely obsolet

Re: Fix typos and inconsistencies for HEAD (take 6)

2019-07-15 Thread Michael Paquier
On Sun, Jul 14, 2019 at 08:24:01AM +0300, Alexander Lakhin wrote: > 6.10. fildes -> fd Not sure that this one was worth bothering. And the rest looks correct after review, so applied! Thanks! > As a side note, while looking at dt_common.c (fixing 6.47), I've got a > feeling that the datetktbl i

Re: Fix typos and inconsistencies for HEAD (take 5)

2019-07-07 Thread Michael Paquier
On Sun, Jul 07, 2019 at 08:03:01AM +0300, Alexander Lakhin wrote: > 5.8. dictlexize -> thesaurus_lexize There could be other dictionaries. > 5.9. regression.diffsregression.planregress/inh -> regression.diffs > planregress/diffs.inh I am wondering if we should not just nuke that... For now I ha

Re: Fix typos and inconsistencies for HEAD

2019-06-30 Thread Michael Paquier
On Sun, Jun 30, 2019 at 04:06:47PM +0300, Alexander Lakhin wrote: > 4.33. close_ - > closept_ This one is incorrect as it refers to the various close_* routines below. > 4.36. combinedproj -> remove (orphaned after 69c3936a) This looks intentional? > I've split proposed patch to make the fixes