Re: [gnome-db] [PATCH] support for inner queries alias without the `AS`.

[Vivien Malerba <vmalerba gmail com>]

Patch committed! Thanks for that correction!

Regards,

Vivien

2016-02-04 4:08 GMT+01:00 Arthur Neves <arthurnn gmail com>:

Thanks Daniel,

I created an issue, and submitted the patch file there too https://bugzilla.gnome.org/show_bug.cgi?id=761529

Vivien, let me know what you think.

cheers,

On Wed, Feb 3, 2016 at 6:48 PM Daniel Espinosa <esodan gmail com> wrote:

May be for Vivien this is Ok.

But may you want to know when your patch is accepted or reviewed. If so ma is better if you file a bug in bugzilla.gnome.org

El feb. 3, 2016 3:39 PM, "Arthur Neves" <arthurnn gmail com> escribió:

Hi folks,

I am using libgda to parse all queries on my app, however I saw that sub-queries that had an alias were not working, for instance: 

  SELECT foo.abc FROM (select abc from bar) foo;

See this is valid on sqlite3:

sqlite> create table bar(abc varchar(10));

sqlite> insert into bar values('hi');

sqlite> SELECT foo.abc FROM (select abc from bar) foo;

hi

Seems like that syntax was not handled by the parser.y lemon file. So I have a patch for it: 

https://github.com/arthurnn/libgda/commit/0ba0e8f49f90304256e4fca637928d8cad55a1e0

Let me know what is the proper way to contribute to this library, if sending a Pull Request on GitHub, or sending a .patch file on the mailing list.

Let me know if this makes sense

Thanks a lot

_______________________________________________
gnome-db-list mailing list
gnome-db-list gnome org
https://mail.gnome.org/mailman/listinfo/gnome-db-list

_______________________________________________
gnome-db-list mailing list
gnome-db-list gnome org
https://mail.gnome.org/mailman/listinfo/gnome-db-list