Re: [gnome-db] refactoring completed

[Rodrigo Moya <rodrigo gnome-db org>]

On Mon, 2002-04-29 at 13:11, Andrew Hill wrote:
> 
> >>So Gonzalo was thinking on using a GScanner-based parser, which will
> >>allow us to know exactly where the syntax error is, thus being more
> >>powerful for our needs.
> >>
> >
> >	Well, the flex/bison parser can tell where the error is and we
> >	can send a error to the caller. Also, if XQL does not check the
> >	syntax of the query, this check can be done converting it to
> >	SQL.
> >
> >	The question is: do we want to be able to report an SQL syntax
> >	error before sending the query to the backend?
> >
> >		* yes: then we need a bison/flex parser. As I said
> >		above, we can start using Andrew's and complete it to
> >		support SQL99.
> >
> >		* no: we can do it with a lexer and some glue.
> >
> 
> Do you want a seperate library for it?  at the moment its a seperate module
> in my code but still in the makefile.
> 
I think we should just have a GdaSqlParser object in libgda. There's no
need for a separated library, IMO.

> i normally use
> module_file_functionname(...);
> GLib variable names are used.
> 
> Oh and someone else (Jean-Michel) asked about database specific sql 
> commands.
> Yeah I find at the moment using custom functions etc also chocks the SQL 
> parser
> so i jsut told my code to try anyway at the query if it fails the sql 
> parser.  Cuase
> it often did on those nasty sql statements.  And mark it that this sql 
> isn't supported
> by the parser.
> 
I think for the time being we are safe in only supporting standard SQL.

> 
> 
> Anyway I'll be writing hte code over next 3 weeks, to get it up to 
> scratch.  I only
> SQL92 but if your familiar with lex your welcome to add in more.  
> 
cool! Having the SQL parser will allow us to add a lot of new features
to libgda/libgnomedb/gnome-db, so, please, just tell us how many beers
you need until you finish the parser, and we'll send them to you :-)

cheers
-- 
Rodrigo Moya <rodrigo gnome-db org> - <rodrigo ximian com>
http://www.gnome-db.org/ - http://www.ximian.com/