Message-ID: <553FA477.5010803@pgmasters.net>
Date: Tue, 28 Apr 2015 11:17:11 -0400
From: David Steele <david@pgmasters.net>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10;
 rv:31.0) Gecko/20100101 Thunderbird/31.6.0
MIME-Version: 1.0
To: Sawada Masahiko <sawada.mshk@gmail.com>,
 Simon Riggs <simon@2ndquadrant.com>
CC: Tatsuo Ishii <ishii@postgresql.org>,
 PostgreSQL-development <pgsql-hackers@postgresql.org>,
 Abhijit Menon-Sen <ams@2ndquadrant.com>,
 Fujii Masao <masao.fujii@gmail.com>
Subject: Re: Auditing extension for PostgreSQL (Take 2)
References: <552326EE.4060509@pgmasters.net>
 <CA+U5nMJkT0ZR3ZD-NhqX=OAfpMJak2ORUE4q=3+zFHay6yuTNw@mail.gmail.com>
 <552D2C1A.3060002@pgmasters.net>
 <20150415.081354.1167351714960358354.t-ishii@sraoss.co.jp>
 <552DA959.2010102@pgmasters.net>
 <CAD21AoDUG8G90SUo70JL=AKDA4dXGNb_wAc4-MvxtwvuMRO0kA@mail.gmail.com>
 <CAD21AoCdr6R7T-PuAQf7S70c6qqX1JvBV55idbfcPtz_tEZ5hQ@mail.gmail.com>
 <552EA121.1090603@pgmasters.net>
 <CAD21AoBoS6Q5yjB05LpfKTzhKmarqwwVq06fuPDFsm0T8gSk_Q@mail.gmail.com>
 <5534FC6E.9030906@pgmasters.net>
 <CAD21AoCxFOLWk-k179PyFxzwjkOW8oPaNk73GxZi+Yw2+n4YFA@mail.gmail.com>
 <553938A1.7030707@pgmasters.net>
 <CAD21AoDBt44ABM_0WMketM8bZtqLBMrJpd9RD5tE=Dkr57ft_A@mail.gmail.com>
In-Reply-To: 
 <CAD21AoDBt44ABM_0WMketM8bZtqLBMrJpd9RD5tE=Dkr57ft_A@mail.gmail.com>
Content-Type: multipart/signed; micalg=pgp-sha256;
 protocol="application/pgp-signature";
 boundary="7hVVgduFQnqkrSmR3V22vPeADXkmm7O2S"
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--7hVVgduFQnqkrSmR3V22vPeADXkmm7O2S
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

On 4/28/15 2:14 AM, Sawada Masahiko wrote:
> On Fri, Apr 24, 2015 at 3:23 AM, David Steele <david@pgmasters.net> wro=
te:
>> I've also added some checking to make sure that if anything looks funn=
y
>> on the stack an error will be generated.
>>
>> Thanks for the feedback!
>>
>=20
> Thank you for updating the patch!
> I ran the postgres regression test on database which is enabled
> pg_audit, it works fine.
> Looks good to me.
>=20
> If someone don't have review comment or bug report, I will mark this
> as "Ready for Committer".

Thank you!  I appreciate all your work reviewing this patch and of
course everyone else who commented on, reviewed or tested the patch
along the way.

--=20
- David Steele
david@pgmasters.net


--7hVVgduFQnqkrSmR3V22vPeADXkmm7O2S
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJVP6R3AAoJEIA2uIJQ5SFAFLkP/1v87Uzh12cqqO/Wx9yBx8Uk
O3nZVhjbzwmIi/sOliJgqPbX4se5r04MdpZTXXjKrVzO75sOA+RkizDT/WO7bHd6
OEW7SEtv0jtAq9+WIKNyZAGtXXniCY8hVtq7zRV6L85/o/KZ1i05/CHSZMwrsduF
aGFcQhB2qZ5oQGFXMGYmXUkGGzANZ4Qo9MS3y0U2Yo2j9o3xs9qwkqPh0QyrIwrF
TN/on5qPLDCPOXqjzdczOPo7w2Hwz2X1xL0BqIjTAPWZPdz4NSyDJqVOJ5ABTqwa
Sv8GgXHPLTUD037hGKUWTcHO/u+T1JR/IeEQmO3oK5niWyoQ2phFVT5MWqKhTiuz
hhcY+MmVNelXQqF1Vrk/aMntAgyRnA5UTxZe+33ZEyezCHRcXclaMLmss8Y3o7Wg
bV92KIXxmSviu6ZsyKFXJGb/MrSNWMyeaaCrNQHBcnqhnxSK1sIi5efoXaFmoNPi
dm9G4thLv0cfkZ5MfSe1BKR6MwkXBUVIxcyNMrlXEtQS50o40osKVQ9p+TfQO35d
2u/KSUL1yGDx3c6r+qf6H60j7d6UWvRR3kFJ4ZfGHAL6J44+QlB2hUHMufQFQw9w
AvAR5xAljITWf0xedo5zvjdnXb/du7CMhFSHfo/5di8a7RI0LWgtbwA6hjzpD92p
SyITjQS1pWRLuCHhmQWT
=/oP+
-----END PGP SIGNATURE-----

--7hVVgduFQnqkrSmR3V22vPeADXkmm7O2S--