Just trying to understand the code here; some very minor comments as I
go along.
partition_op_strategy returning int32 looks pretty ugly, and the calling
code is not super-intelligible either. How about returning a value from
a new enum?
typedef PartClause is missing a struct name, as is our tradition.
> + * We don't a <> operator clause into a key right away.
Missing a word there.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services