* Kevin Grittner (kgrittn@ymail.com) wrote:
> That's the point, and the whole point. You have not shown that it
> doesn't. You have not shown why adding a 12th non-default opclass
> is a particular problem here (although we have a consensus to use
> different operators, to reserve this operator namespace for other
> things).
We need justification to add operators, imv, especially ones that expose
our internal binary representation of data. I worry that adding these
will come back to bite us later and that we're making promises we won't
be able to keep.
If these inconsistencies in what happens with these data types are an
issue then REFRESH can be handled as a wholesale DELETE/INSERT. Trying
to do this incremental-but-not-really maintenance where the whole query
is run but we try to skimp on what's actually getting updated in the
matview is a premature optimization, imv, and one which may be less
performant and more painful, with more gotchas and challenges for our
users, to deal with in the long run.
Thanks,
Stephen
