Fix XXX comment about first elements of jsonb stavalues

Nikita Glukhov · Nikita Glukhov · commit 4d18bfbfe423 · 2022-03-11T00:52:05.000+03:00
diff --git a/src/backend/utils/adt/jsonb_typanalyze.c b/src/backend/utils/adt/jsonb_typanalyze.c
@@ -35,14 +35,15 @@
  *
  * This is stored in the stavalues array.
  *
+ * The first element of stavalues is a path prefix.  It is used for avoiding
+ * path transformations when the derived statistics for the chains of ->
+ * operators is computed.
+ *
  * The per-column stats (stored in the "json" key) have additional internal
  * structure, to allow storing multiple stakind types (histogram, mcv). See
  * jsonAnalyzeMakeScalarStats for details.
  *
  *
- * XXX There's additional stuff (prefix, length stats) stored in the first
- * two elements, I think.
- *
  * XXX It's a bit weird the "regular" stats are stored in the "json" key,
  * while the JSON stats (frequencies of different JSON types) are right
  * at the top level.

Original file line number	Diff line number	Diff line change
`@@ -35,14 +35,15 @@`
`35`	`35`	`*`
`36`	`36`	`* This is stored in the stavalues array.`
`37`	`37`	`*`
	`38`	`+ * The first element of stavalues is a path prefix. It is used for avoiding`
	`39`	`+ * path transformations when the derived statistics for the chains of ->`
	`40`	`+ * operators is computed.`
	`41`	`+ *`
`38`	`42`	`* The per-column stats (stored in the "json" key) have additional internal`
`39`	`43`	`* structure, to allow storing multiple stakind types (histogram, mcv). See`
`40`	`44`	`* jsonAnalyzeMakeScalarStats for details.`
`41`	`45`	`*`
`42`	`46`	`*`
`43`		`- * XXX There's additional stuff (prefix, length stats) stored in the first`
`44`		`- * two elements, I think.`
`45`		`- *`
`46`	`47`	`* XXX It's a bit weird the "regular" stats are stored in the "json" key,`
`47`	`48`	`* while the JSON stats (frequencies of different JSON types) are right`
`48`	`49`	`* at the top level.`