On 2018-06-05 19:04:11 +0200, David Fetter wrote: > On Tue, Jun 05, 2018 at 02:56:23PM +1200, David Rowley wrote: > > On 5 June 2018 at 06:52, Andres Freund <and...@anarazel.de> wrote: > > > That part has gotten a bit easier since, because we have serialize > > > / deserialize operations for aggregates these days. > > > > True. Although not all built in aggregates have those defined. > > Just out of curiosity, which ones don't? As of > 3f85c62d9e825eedd1315d249ef1ad793ca78ed4, pg_aggregate has both of > those as NOT NULL.
That doesn't mean much. We commonly store 0 / InvalidOid for such things in the catalog. Allows us to still map the tuples to structs (due to fixed width). Greetings, Andres Freund
