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

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

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

Fix typos and inconsistencies for HEAD (take 11)

2019-08-18 Thread Alexander Lakhin
Hello hackers, Please consider fixing the last group (for now) of typos and inconsistencies in the tree: 11.1 peforming -> performing 11.2 table_freeze_age -> freeze_table_age 11.3 TableInfoData -> TableDataInfo 11.4 TableStatus -> PgStat_TableStatus 11.5 TAG_VALID -> BM_TAG_VALID 11.6

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

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 ->

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

2019-08-05 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

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

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

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.

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

Fix typos and inconsistencies for HEAD (take 9)

2019-08-04 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next truss of typos and inconsistencies in the tree: 9.1. NAMESPACE_SQLXML -> remove (not used since the introduction in 355e05ab) 9.2. NBXLOG_H -> NBTXLOG_H 9.3. NEWPAGE -> XLOG_FPI (orphaned since 54685338) 9.4. newXlogId, newXlogSeg -> newXlogSegNo

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

Fix typos and inconsistencies for HEAD (take 8)

2019-07-27 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next set of typos and inconsistencies in the tree: 8.1. LABORT -> LIKE_ABORT 8.2. LagTrackerWriter -> LagTrackerWrite 8.3. lag_with_offset_and_default, * -> window_lag_with_offset_and_default, window_* (in windowfuncs.c) 8.4. language-name ->

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 >>

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

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

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

Fix typos and inconsistencies for HEAD (take 7)

2019-07-20 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next pack of typos and inconsistencies in the tree: 7.1. h04m05s06 -> h04mm05s06 (in fact it's broken since 6af04882, but h04mm05s06.789 still accepted) 7.2. hasbucketcleanup -> hashbucketcleanup 7.3. _hashm_spare -> hashm_spares 7.4. hashtbl -> hash

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

Fix typos and inconsistencies for HEAD (take 6)

2019-07-13 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next batch of typos and inconsistencies in the tree: 6.1. FADVISE_WILLNEED -> POSIX_FADV_WILLNEED 6.2. failOK -> missing_ok 6.3. failOnerror -> failOnSignal 6.4. fakebits -> remove (irrelevant since introduction in 945543d9) 6.5. FastPathGetLockEntry ->

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

Fix typos and inconsistencies for HEAD (take 5)

2019-07-06 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next cluster of typos and inconsistencies in the tree: 5.1. datetkntbl -> datetktbl 5.2. datminxmid -> datminmxid 5.3. DatumGetP -> DatumGetPointer 5.4. ECPG_COMPILE, DECPG_COMPILE -> remove (orphaned since 328d235) 5.5. defer_cleanup_age ->

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

Fix typos and inconsistencies for HEAD

2019-06-30 Thread Alexander Lakhin
Hello hackers, Please consider fixing the next bunch of typos and inconsistencies in the tree: 4.1. AccesShareLock -> AccessShareLock 4.2. AdjustTimestampForTypemod -> AdjustTimestampForTypmod 4.3. AExprConst -> AexprConst 4.4. AlterExtensionOwner_oid - remove (orphaned after 994c36e0) 4.5.