Merge lp://staging/~jamesh/storm/bug-585704 into lp://staging/storm

Seems reasonable, +1!

After applying your fix, we are getting this:

ERROR: operator does not exist: lifecycle_status = bytea
LINE 1: ...th = E'~/Ubuntu One' AND UserDefinedFolder.status = E'Live':...

When we use a Type:

CREATE TYPE lifecycle_status as enum('Live', 'Dead')

> After applying your fix, we are getting this:
>
> ERROR: operator does not exist: lifecycle_status = bytea
> LINE 1: ...th = E'~/Ubuntu One' AND UserDefinedFolder.status = E'Live':...
>
> When we use a Type:
>
> CREATE TYPE lifecycle_status as enum('Live', 'Dead')

BTW, In order to fix this, we made sure we only assigned Unicode strings to the Properties.

John: I assume you'd get that error without my fix if you move to the new version of psycopg2.

This branch doesn't change the way Storm passes byte strings down to psycopg2 (it has always wrapped them in psycopg2.Binary() objects) -- it just fixes a few cases inside Storm where we got byte string vs. unicode string wrong. Switching those custom variable types to use unicode strings was the correct fix for the U1 code.

thanks for the reply, this fix worked great for us.