It gets even better. I put together some code to determine driver behavior and it seems to work. The problem is that now I'm testing against PG 8.0, which I just found out doesn't support the "lastval()" function :(. This begs the question, can we document somewhere what versions of the databases that we support? I could do it in Scaladoc on the DriverTypes classes, but I'd at least like to write somewhere that we only support versions X, Y and Z. If we want to support PG 8.0 I'm going to have to figure out how to get the generated value some other way...
Derek On Thu, Aug 20, 2009 at 3:53 PM, Derek Chen-Becker <[email protected]>wrote: > I should have a real fix out pretty soon that will take into account the > server version. Actually, the "current" behavior of calling lastval() is > broken if columns other than the primary key are auto-generated. I think > that this is pretty rare but I'd still like to look into fixing it. > > Derek > > > On Thu, Aug 20, 2009 at 4:47 PM, David Pollak < > [email protected]> wrote: > >> >> >> On Thu, Aug 20, 2009 at 2:31 PM, Derek Chen-Becker <[email protected] >> > wrote: >> >>> A fix has been pushed to master and 1.0_maint. Give hudson some time to >>> build 1.1-SNAPSHOT and I'll push a 1.0.1-SNAPSHOT release from here. >>> Apologies for the issue. >> >> >> Thanks for your wicked fast response! >> >> >>> >>> >>> Derek >>> >>> >>> On Thu, Aug 20, 2009 at 4:08 PM, David Pollak < >>> [email protected]> wrote: >>> >>>> Folks, >>>> >>>> We changed the way that the PostgreSQL retrieved primary keys from >>>> INSERTs. Turns out that the new code does not work with PG 8.1. >>>> >>>> You will see something like: >>>> >>>> ERROR: syntax error at or near "RETURNING" >>>> org.postgresql.util.PSQLException: ERROR: syntax error at or near >>>> "RETURNING" >>>> at >>>> org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:1559) >>>> at >>>> org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:1327) >>>> at >>>> org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:193) >>>> at >>>> org.postgresql.jdbc2.AbstractJdbc2Statement.execute(AbstractJdbc2Statement.java:452) >>>> at >>>> org.postgresql.jdbc2.AbstractJdbc2Statement.executeWithFlags(AbstractJdbc2Statement.java:351) >>>> at >>>> org.postgresql.jdbc2.AbstractJdbc2Statement.executeQuery(AbstractJdbc2Statement.java:255) >>>> at >>>> net.liftweb.mapper.PostgreSqlDriver$.performInsertWithPK(Driver.scala:207) >>>> at net.liftweb.mapper.DriverType.performInsert(Driver.scala:68) >>>> at >>>> net.liftweb.mapper.MetaMapper$$anonfun$12.apply(MetaMapper.scala:676) >>>> at >>>> net.liftweb.mapper.MetaMapper$$anonfun$12.apply(MetaMapper.scala:622) >>>> >>>> If you are running PG 8.1 and Lift. We will be fixing the problem ASAP. >>>> >>>> Thanks, >>>> >>>> David >>>> >>>> -- >>>> Lift, the simply functional web framework http://liftweb.net >>>> Beginning Scala http://www.apress.com/book/view/1430219890 >>>> Follow me: http://twitter.com/dpp >>>> Git some: http://github.com/dpp >>>> >>>> >>>> >>> >>> >>> >> >> >> -- >> Lift, the simply functional web framework http://liftweb.net >> Beginning Scala http://www.apress.com/book/view/1430219890 >> Follow me: http://twitter.com/dpp >> Git some: http://github.com/dpp >> >> >> >> > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Lift" 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/liftweb?hl=en -~----------~----~----~----~------~----~------~--~---
