OK, this is pushed into 1.1-SNAPSHOT and 1.0.1-SNAPSHOT. On Fri, Aug 21, 2009 at 11:50 AM, David Pollak < [email protected]> wrote:
> > > On Fri, Aug 21, 2009 at 9:02 AM, Derek Chen-Becker > <[email protected]>wrote: > >> 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? > > > That would be good. Also, if we can fail early by throwing an exception > with a very detailed message, that'd be great! > > >> 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... > > > Maybe on a wiki page. > > Thanks for your diligent work on this! > > David > > >> >> >> 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 >>>> >>>> >>>> >>> >> >> >> > > > -- > 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 -~----------~----~----~----~------~----~------~--~---
