Re: Event Triggers reduced, v1

Dimitri Fontaine <dimitri@2ndquadrant.fr>

From: Dimitri Fontaine <dimitri@2ndQuadrant.fr>

To: Dimitri Fontaine <dimitri@2ndQuadrant.fr>

Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>

Date: 2012-07-04T17:56:48Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Replace int2/int4 in C code with int16/int32
- b8b2e3b2deea 9.3.0 cited

Attachments

event_triggers_v1.3--v1.5.patch (text/x-patch) patch v1
event_triggers_v1.5.patch.gz (application/octet-stream) patch v1

Dimitri Fontaine <dimitri@2ndQuadrant.fr> writes:
> Robert Haas <robertmhaas@gmail.com> writes:
>> No, I'm not asking you to add any more columns right now (in fact,
>> please do not).  But the type of the existing column should change to
>> text[].
>
> Ok, done in the attached. We now read text from either the user input in
> the grammar or from the catalogs (in a 1-D array there), and convert to
> our enum structure for internal code use.

In the previous one I missed adapting pg_dump and psql, here's the
updated set of patches. Sorry about that. This unexpected electrical
shut down in the middle of the day was not cool. Nor was the second one.

Regards,
-- 
Dimitri Fontaine
http://2ndQuadrant.fr     PostgreSQL : Expertise, Formation et Support