ops, I'm back. well, sqlite doesn't support :limit anyways, so I guess we should skip this test for adapters that doesn't support it, correct?
any ideas? On 7/6/07, Michael Koziarski <[EMAIL PROTECTED]> wrote: > > > Well, I was asked to include #5484 into this ticket, so it is dealing > > with distinct bugs at once. > > Sounds good. This patch breaks sqlite's tests, if you can merge > those in, it seems good to apply. > > > > To understand those bugs, let's begin with a single migration that > > creates a table with one column: > > t.column :limit3, :integer, :limit => 3 > > > > 1) When you run this migration under postgresql, the column is dumped > > to schema.rb without the limit: > > t.column "limit3", :integer > > > > The patch in this ticket fixes the postgresql adapter to dump columns > > with limits. > > > > 2) Also in postgresql, the above migration will generate a smallint > > column, which uses 2 bytes for storage. I think it should generate an > > integer column, so it can handle 3 bytes numbers without overflow -- > > and this is also covered by the patch attached. > > > > 3) Finally, in mysql the migration will generate a INT(3) column, what > > means a column with display size of 3, and not a column with 3 bytes > > for storage. The patch changes mysql adapter to use the limit > > constraint to generate smallint, mediumint, int and bigint columns > > accordingly. > > > > > > On 7/3/07, Michael Koziarski <[EMAIL PROTECTED]> wrote: > > > > > > > Can anyone check this ticket? > > > > http://dev.rubyonrails.org/ticket/5489 > > > > > > > > It's 1 year old already, but still valid imho. > > > > > > I don't quite follow what the problem is with the current > > > implementation? could you give me an example of what bugs / > > > mis-behaviour this patch will prevent? > > > > > > > > > -- > > > Cheers > > > > > > Koz > > > > > > > > > > > > > > > > > > > > -- > Cheers > > Koz > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" 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/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
