Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1anXlT-0001Xa-N7 for pgsql-hackers@arkaria.postgresql.org; Tue, 05 Apr 2016 20:39:55 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1anXlT-0003KV-71 for pgsql-hackers@arkaria.postgresql.org; Tue, 05 Apr 2016 20:39:55 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1anXlS-0003KP-RC for pgsql-hackers@postgresql.org; Tue, 05 Apr 2016 20:39:54 +0000 Received: from linode.thelabyrinth.net ([198.58.117.137] helo=mail.thelabyrinth.net) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1anXlP-0005hF-Hs for pgsql-hackers@postgresql.org; Tue, 05 Apr 2016 20:39:54 +0000 Received: from desktop.home.thelabyrinth.net (unknown [98.204.177.190]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: dsteele) by mail.thelabyrinth.net (Postfix) with ESMTPSA id C47A912CFD; Tue, 5 Apr 2016 20:39:48 +0000 (UTC) Subject: Re: [PATCH v12] GSSAPI encryption support To: Michael Paquier , Robbie Harwood References: Cc: PostgreSQL mailing lists From: David Steele Message-ID: <57042294.70308@pgmasters.net> Date: Tue, 5 Apr 2016 16:39:48 -0400 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0) Gecko/20100101 Thunderbird/38.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Pg-Spam-Score: -1.9 (-) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-hackers Precedence: bulk Sender: pgsql-hackers-owner@postgresql.org On 4/5/16 1:25 AM, Michael Paquier wrote: > Btw, those seem like small things to me, and my comments have been > addressed, so I have switched the patch as ready for committer. I > guess that Stephen would be the one to look at it. I have also run this patch through my tests and didn't find any problems. I agree that it is ready for committer. I don't see a problem with using the top memory context but whoever commits it may feel differently. I'm happy to leave that decision up to them. -- -David david@pgmasters.net -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers