ticket #1146 completed in r5056, r5057. (0.5/0.4)
On Aug 20, 2008, at 4:31 PM, Ross Vandegrift wrote:
>
> Hey Mike,
>
> I have a DB that uses MySQL's MEDIUMINT type in a bunch of places.
> Because many of those are used as foreign keys, I need SQLAlchemy to
> be happy emitting tables with MEDIUMINT types. The below patch does
> that. This is against an 0.4 checkout from a few mins ago.
>
>
> Index: lib/sqlalchemy/databases/mysql.py
> ===================================================================
> --- lib/sqlalchemy/databases/mysql.py (revision 5048)
> +++ lib/sqlalchemy/databases/mysql.py (working copy)
> @@ -172,7 +172,7 @@
> 'MSMediumBlob', 'MSMediumText', 'MSNChar', 'MSNVarChar',
> 'MSNumeric', 'MSSet', 'MSSmallInteger', 'MSString', 'MSText',
> 'MSTime', 'MSTimeStamp', 'MSTinyBlob', 'MSTinyInteger',
> - 'MSTinyText', 'MSVarBinary', 'MSYear' )
> + 'MSTinyText', 'MSVarBinary', 'MSYear', 'MSMediumInteger' )
>
>
> RESERVED_WORDS = util.Set(
> @@ -549,6 +549,33 @@
> return self._extend("BIGINT")
>
>
> +class MSMediumInteger(MSInteger):
> + """MySQL MEDIUMINTEGER type."""
> +
> + def __init__(self, length=None, **kw):
> + """Construct a MEDIUMINTEGER
> +
> + length
> + Optional, maximum display width for this number.
> +
> + unsigned
> + Optional.
> +
> + zerofill
> + Optional. If true, values will be stored as strings left-
> padded with
> + zeros. Note that this does not effect the values returned
> by the
> + underlying database API, which continue to be numeric.
> + """
> +
> + super(MSMediumInteger, self).__init__(length, **kw)
> +
> + def get_col_spec(self):
> + if self.length is not None:
> + return self._extend("MEDIUMINT(%(length)s)" %
> {'length': self.length})
> + else:
> + return self._extend("MEDIUMINT")
> +
> +
> class MSTinyInteger(MSInteger):
> """MySQL TINYINT type."""
>
>
> --
> Ross Vandegrift
> [EMAIL PROTECTED]
>
> "The good Christian should beware of mathematicians, and all those who
> make empty prophecies. The danger already exists that the
> mathematicians
> have made a covenant with the devil to darken the spirit and to
> confine
> man in the bonds of Hell."
> --St. Augustine, De Genesi ad Litteram, Book II, xviii, 37
>
> >
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sqlalchemy" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/sqlalchemy?hl=en
-~----------~----~----~----~------~----~------~--~---
