Sorry, dropped the ball on this one. Maybe change something in
MySqlDriver, perhaps primaryKeySetup? Unfortunately at that level you
don't know what columns have been defined. Anyway, I don't think this
is a pressing issue, so in the absence of having a solution to offer,
I'll leave it be.

Peter

On Dec 16, 12:55 pm, David Pollak <feeder.of.the.be...@gmail.com>
wrote:
> On Wed, Dec 16, 2009 at 11:07 AM, Peter Robinett 
> <pe...@bubblefoundry.com>wrote:
>
> > Fair enough, but isn't this still a non-optimal situation with the
> > MySQL database driver?
>
> What is your proposed change?
>
>
>
>
>
>
>
> > Peter
>
> > On Dec 15, 7:23 pm, David Pollak <feeder.of.the.be...@gmail.com>
> > wrote:
> > > On Tue, Dec 15, 2009 at 5:46 PM, Peter Robinett <pe...@bubblefoundry.com
> > >wrote:
>
> > > > My tables created by Schemefier have multiple keys for the primary key
> > > > when my Mapper model is declared like so: class myModel extends
> > > > LongKeyedMapper[myModel] with IdPK. There is naturally the primary key
> > > > but there is also a unique key on the column that simply duplicates
> > > > the functionality of the primary key.
>
> > > > Are other people seeing this? I've only looked on MySQL. This seems to
> > > > be because IdPK adds an id of type MappedLongIndex. Only the
> > > > longIndexColumnType of MySqlDriver has "UNIQUE KEY" in its definition.
> > > > Unique keys are great but, as I mentioned, if the column is also the
> > > > primary key the index is a duplication. Maybe this is something that
> > > > IdPK can disable when declaring the column? At least with MySQL
> > > > primary keys are guaranteed to be unique and InnoDB tables, which
> > > > Mapper uses, can have issues with long primary keys[1].
>
> > > > Should this be fixed? Or is the more helpful answer 'Don't use
> > > > MySQL'? ;-)
>
> > > I would use MySQL for a production site.
>
> > > If IdPK isn't doing what you want, then don't use it, just declare the
> > > primary key field manually.
>
> > > > Peter
>
> > > > [1]:http://dev.mysql.com/doc/refman/5.1/en/innodb-index-types.html
>
> > > > --
>
> > > > You received this message because you are subscribed to the Google
> > Groups
> > > > "Lift" group.
> > > > To post to this group, send email to lift...@googlegroups.com.
> > > > To unsubscribe from this group, send email to
> > > > liftweb+unsubscr...@googlegroups.com<liftweb%2bunsubscr...@googlegroups.com
> > > >  >
> > <liftweb%2bunsubscr...@googlegroups.com<liftweb%252bunsubscr...@googlegroup 
> > s.com>>
> > > > .
> > > > For more options, visit this group at
> > > >http://groups.google.com/group/liftweb?hl=en.
>
> > > --
> > > Lift, the simply functional web frameworkhttp://liftweb.net
> > > Beginning Scalahttp://www.apress.com/book/view/1430219890
> > > Follow me:http://twitter.com/dpp
> > > Surf the harmonics
>
> > --
>
> > You received this message because you are subscribed to the Google Groups
> > "Lift" group.
> > To post to this group, send email to lift...@googlegroups.com.
> > To unsubscribe from this group, send email to
> > liftweb+unsubscr...@googlegroups.com<liftweb%2bunsubscr...@googlegroups.com 
> > >
> > .
> > For more options, visit this group at
> >http://groups.google.com/group/liftweb?hl=en.
>
> --
> Lift, the simply functional web frameworkhttp://liftweb.net
> Beginning Scalahttp://www.apress.com/book/view/1430219890
> Follow me:http://twitter.com/dpp
> Surf the harmonics

--

You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to lift...@googlegroups.com.
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en.


Reply via email to