[9.4 CF 1] Patches which desperately need code review

Lists: pgsql-hackers
From: Josh Berkus <josh(at)agliodbs(dot)com>
To: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: [9.4 CF 1] Patches which desperately need code review
Date: 2013-07-18 21:31:11
Message-ID: 51E85E9F.2060906@agliodbs.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hackers,

The following three patches really really need some code review love.
Until they get a code review, we can't close out the CommitFest:

Row-Level Security:
https://commitfest.postgresql.org/action/patch_view?id=874

Revive Line Type:
https://commitfest.postgresql.org/action/patch_view?id=1154

Performance Improvement by reducing WAL for Update Operation:
https://commitfest.postgresql.org/action/patch_view?id=984

Thanks!

--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Josh Berkus <josh(at)agliodbs(dot)com>
Cc: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [9.4 CF 1] Patches which desperately need code review
Date: 2013-07-18 22:15:07
Message-ID: 20130718221507.GC4139@eldon.alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Josh Berkus wrote:

> Revive Line Type:
> https://commitfest.postgresql.org/action/patch_view?id=1154

This one's easy -- we're waiting on a decision on whether to use A,B,C
text representation. Honestly, it seems a no-brainer to me that this is
what it should use; the other representation seems to have too many
problems.

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services