Fix a couple typos in BRIN code
authorTomas Vondra
Fri, 19 Apr 2024 13:43:14 +0000 (15:43 +0200)
committerTomas Vondra
Fri, 19 Apr 2024 13:43:17 +0000 (15:43 +0200)
Typos introduced by commits c1ec02be1d79b43757171470 and dae761a87eda.

Author: Alvaro Herrera
Reported-by: Alexander Lakhin
Discussion: https://postgr.es/m/202401091043[email protected]

doc/src/sgml/indexam.sgml
src/backend/access/brin/brin.c

index 76ac0fcddd78d6da0ae656e68e807947e2196865..18cf23296f2b310f136033ea4088fbb11dc6439a 100644 (file)
@@ -370,7 +370,7 @@ aminsert (Relation indexRelation,
    initially). After the index insertions complete, the memory will be freed
    automatically. If additional cleanup is required (e.g. if the cache contains
    buffers and tuple descriptors), the AM may define an optional function
-   indexinsertcleanup, called before the memory is released.
+   aminsertcleanup, called before the memory is released.
   
 
   
index 32722f0961b3128112ecbbc85ec38e3e9b767640..4f708bba658ec17825d3b27cf3b78d1dcc11a4f8 100644 (file)
@@ -222,7 +222,7 @@ static bool add_values_to_range(Relation idxRel, BrinDesc *bdesc,
                                BrinMemTuple *dtup, const Datum *values, const bool *nulls);
 static bool check_null_keys(BrinValues *bval, ScanKey *nullkeys, int nnullkeys);
 static void brin_fill_empty_ranges(BrinBuildState *state,
-                                  BlockNumber prevRange, BlockNumber maxRange);
+                                  BlockNumber prevRange, BlockNumber nextRange);
 
 /* parallel index builds */
 static void _brin_begin_parallel(BrinBuildState *buildstate, Relation heap, Relation index,
@@ -1151,8 +1151,8 @@ brinbuild(Relation heap, Relation index, IndexInfo *indexInfo)
     *
     * XXX plan_create_index_workers makes the number of workers dependent on
     * maintenance_work_mem, requiring 32MB for each worker. That makes sense
-    * for btree, but not for BRIN, which can do away with much less memory.
-    * So maybe make that somehow less strict, optionally?
+    * for btree, but not for BRIN, which can do with much less memory. So
+    * maybe make that somehow less strict, optionally?
     */
    if (indexInfo->ii_ParallelWorkers > 0)
        _brin_begin_parallel(state, heap, index, indexInfo->ii_Concurrent,
@@ -2621,7 +2621,7 @@ _brin_parallel_merge(BrinBuildState *state)
 
    /*
     * Initialize BrinMemTuple we'll use to union summaries from workers (in
-    * case they happened to produce parts of the same paga range).
+    * case they happened to produce parts of the same page range).
     */
    memtuple = brin_new_memtuple(state->bs_bdesc);
 
@@ -2932,7 +2932,7 @@ _brin_parallel_build_main(dsm_segment *seg, shm_toc *toc)
  * specified block number. The empty tuple is initialized only once, when it's
  * needed for the first time, stored in the memory context bs_context to ensure
  * proper life span, and reused on following calls. All empty tuples are
- * exactly the same except for the bs_blkno field, which is set to the value
+ * exactly the same except for the bt_blkno field, which is set to the value
  * in blkno parameter.
  */
 static void