Takahiro Itagaki wrote:
Can I ask the final decision whether the YAML formatter should be
applied or rejected? As far as I read the discussion, we can apply it
because serveral users want it and we don't have a plan to support
extensible formatters in the core.
The path I thought made sense at this point was to mark the patch ready
for a committer, since it sounds like everyone is done with it now, and
have another committer besides yourself do a final review as part of
that. At this point, I think we've justified the feature and confirmed
the feature works. Given the controversy, I think another set of eyes
to make sure it's not going to be a maintenance headache moving forward
should (as usual) have the final say on whether the code goes in or not,
because that's only drawback to it left to committing it I see at this
point.
To be clear about which version we're talking about:
http://archives.postgresql.org/message-id/20091130123456.4a03.52131...@oss.ntt.co.jp
is the candidate for commit that includes the cleanup you've already done.
--
Greg Smith 2ndQuadrant Baltimore, MD
PostgreSQL Training, Services and Support
g...@2ndquadrant.com www.2ndQuadrant.com
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
