Re: YAML Was: CommitFest status/management

Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>

From: Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>
To: Greg Smith <greg@2ndquadrant.com>
Cc: Dimitri Fontaine <dfontaine@hi-media.com>, Robert Haas <robertmhaas@gmail.com>, Josh Berkus <josh@agliodbs.com>, Tom Lane <tgl@sss.pgh.pa.us>, Ron Mayer <rm_pg@cheapcomplexdevices.com>, Andrew Dunstan <andrew@dunslane.net>, pgsql-hackers@postgresql.org
Date: 2009-12-09T02:26:24Z
Lists: pgsql-hackers
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.

Greg Smith <greg@2ndquadrant.com> wrote:
> -The patch is small to apply
> -Having one more "legacy" format to satisfy would actually improve the 
> odds that a future modular EXPLAIN would be robustly designed
> -While it's questionable whether it's strictly a majority on voting 
> here, it's close, which suggests there is plenty of support for wanting 
> this feature
> -Since nothing is removed the people who aren't in favor of this format 
> aren't negatively impacted by it being committed

Comments from additional discussion:

  - YAML format is the second (or the best for some people)
    human-unreadabe format for EXPLAIN output.

  - JSON is not always a subset of YAML. It is not recommended to
    generate/parse YAML with a JSON generator/parser or vice versa.

  - We won't add any core hooks and plug-in to format EXPLAIN output.
    Client tools may convert the output if needed. (ex. using XLST)

  - The spec of YAML might be changed in the feature, but we use
    only the basic parts. The parts will not be changed.

Regards,
---
Takahiro Itagaki
NTT Open Source Software Center