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

Commit 43cd7ab

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 356f85f commit 43cd7ab

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/backend/access/brin/brin_inclusion.c

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

0 commit comments

Comments
 (0)