Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 530bcf7

Browse files
committed
Fix thinko in comment
The listed numbers disagreed with the ones being used in the symbols; but instead of just fixing the numbers in the comment, use the symbolic name instead, which seems clearer. This has been wrong all along, so apply back to 9.5 where BRIN was introduced. Reported-by: Tomas Vondra Discussion: https://postgr.es/m/5ff514f2-8b1e-6366-b11c-8e2ed442562d@2ndquadrant.com
1 parent 186b6df commit 530bcf7

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/backend/access/brin/brin_inclusion.c

+8-4
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,14 @@
5959
/*-
6060
* The values stored in the bv_values arrays correspond to:
6161
*
62-
* 0 - the union of the values in the block range
63-
* 1 - whether an empty value is present in any tuple in the block range
64-
* 2 - whether the values in the block range cannot be merged (e.g. an IPv6
65-
* address amidst IPv4 addresses).
62+
* INCLUSION_UNION
63+
* the union of the values in the block range
64+
* INCLUSION_UNMERGEABLE
65+
* whether the values in the block range cannot be merged
66+
* (e.g. an IPv6 address amidst IPv4 addresses)
67+
* INCLUSION_CONTAINS_EMPTY
68+
* whether an empty value is present in any tuple
69+
* in the block range
6670
*/
6771
#define INCLUSION_UNION 0
6872
#define INCLUSION_UNMERGEABLE 1

0 commit comments

Comments
 (0)