Merge lp:~allenap/launchpad/switch-to-json-property-bug-845904 into lp:launchpad/db-devel

[Gavin Panella]

This changes the distributionjob.json_data column to BYTEA which is expected by the new storm.properties.JSON column type. This might *require* a simultaneous code change, but I need to think about it some more, and perhaps you have some insight too, or a way around it.

[Robert Collins (lifeless)]

looks like storm is broken to me - json is a text format (which is why
bson exists).

-1 without further examination!

[Robert Collins (lifeless)]

review: Disapprove

Unmerged revisions

10965. By Gavin Panella on 2011-09-09

Change DistributionJob.json_data to a BYTEA column. This may *require* a simultaneous code change.

=== added file 'database/schema/patch-2208-99-1.sql'
--- database/schema/patch-2208-99-1.sql	1970-01-01 00:00:00 +0000
+++ database/schema/patch-2208-99-1.sql	2011-09-09 19:38:25 +0000
@@ -0,0 +1,9 @@
+-- Copyright 2011 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+SET client_min_messages=ERROR;
+
+ALTER TABLE distributionjob
+  ALTER json_data SET DATA TYPE bytea
+  USING convert_to(json_data, 'utf-8');
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2208, 99, 1);