On 9-11-2014 12:01, Mark Rotteveel wrote: > On 22-8-2014 16:54, Mark Rotteveel wrote: >> Are there any objections if I try to come up with a patch to add OFFSET >> and FETCH to the select syntax (CORE-4526)? > > Attached you will find my patch for this feature. > > The grammar matches the SQL definition. I have made the existing > rows_clause required so it doesn't cause conflicts with the new > select_expr alternative that uses the result_offset_clause and > fetch_first_clause. For the delete_searched and update_searched I added > a rows_clause_optional so they continue to work as is. > > I have reused the existing RowsClause node. I was tempted to rename it > to OffsetFetchClause as the implementation actually better matches the > needs for offset/fetch than for rows, but I left it as is. > > I also updated some of the error messages and included a readme file. > > Let me know if I can commit this, or if things need to be further improved.
Anyone? -- Mark Rotteveel ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk Firebird-Devel mailing list, web interface at https://lists.sourceforge.net/lists/listinfo/firebird-devel
