On Friday, March 16, 2012 12:23:18 PM UTC-7, Jeremy Evans wrote: > > On Friday, March 16, 2012 11:57:38 AM UTC-7, Doug Tykocki wrote: >> >> I ran the same bit of code against SQL Server 2005 and got >> a Sequel::DatabaseError: TinyTds::Error: 'ROLLUP' is not a recognized >> built-in function name. >> > > Ah, looks like SQL Server 2005 supports WITH ROLLUP and SQL Server 2008 > supports ROLLUP(), though 2008 also supports WITH ROLLUP for backwards > compatibility. See: > > 2005: > http://technet.microsoft.com/en-us/library/ms177673%28v=sql.90%29.aspx > 2008: http://technet.microsoft.com/en-us/library/ms177673.aspx > > It says that WITH ROLLUP will be removed in a future version of SQL > Server. I'll modify Sequel to use WITH ROLLUP/CUBE instead of > ROLLUP()/CUBE() on SQL Server 2005. > > I only test on SQL Server 2008, which is why I didn't catch this bug. >
This should fix it: https://github.com/jeremyevans/sequel/commit/5aa75e16c1aae0ed56683505f30f755b0a650cfb Can any SQL 2005 users confirm? Thanks, Jeremy -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To view this discussion on the web visit https://groups.google.com/d/msg/sequel-talk/-/bCb3tIEv7_YJ. 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/sequel-talk?hl=en.
