|From:||Abhijit Menon-Sen <ams(at)toroid(dot)org>|
|To:||Andrew Dunstan <andrew(at)dunslane(dot)net>|
|Subject:||Re: JSON for PG 9.2|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
At 2012-01-15 11:08:05 -0500, andrew(at)dunslane(dot)net wrote:
> Here's an update that adds row_to_json, plus a bit more cleanup.
I've started reviewing this patch, but it'll take me a bit longer to go
through json.c properly. Here are a few preliminary notes:
1. The patch has a lot of whitespace errors (primarily lines ending in
whitespace), but applying with git apply --whitespace=fix isn't safe,
because the test results need some (but not all) of those spaces. I
applied the patch, backed out the changes to expected/json.out, and
created the file from the patch, then removed the superfluous
2. I bumped some function OIDs to avoid conflicts.
3. One documentation typo.
Everything other than json.c (which I haven't read yet) looks fine
(builds, passes tests). I've attached a patch covering the changes
|Next Message||Peter Geoghegan||2012-01-27 04:36:19||Re: Progress on fast path sorting, btree index creation time|
|Previous Message||Robert Haas||2012-01-27 03:32:14||Re: Progress on fast path sorting, btree index creation time|