Hi Sergei, I studied lex and yacc and understood complete parser written in sql_yacc.yy. Also went through the constants and symbols declared in lex.h, sql_yacc.h, sql_cmd.h. I found out that OR REPLACE and IF NOT EXISTS is already implemented for tables. The IF NOT EXISTS rule is also defined for index in yacc file but not handled in the execution code. And for other object types we need to add the rules in yacc files and also need to change the code in "mysql_create_" functions in respective files. The OR REPLACE and IF NOT EXISTS are handled in the create execution itself. I found two flags in LEX class which are used for the purpose, create_info.options and check_exists. So, are all the things that I have compiled correct? I have an initial draft of my proposal ready. I will upload it on the GSOC portal today, please review. Should I also email the same to you? Looking forward to your reply. Thanks, Sriram On Tue, Mar 18, 2014 at 9:53 PM, Sergei Golubchik wrote:

Hi, Sriram!

On Mar 18, Sriram Patil wrote:

...

Hi Sergei,

Thank you for such a quick reply. I will look into the files you have mentioned.

Also, is there any particular format for the proposal draft? Or may be if you expect some points to be mentioned clearly in the proposal draft.

No particular format, look at templates of other organizations, if you want something to start from.

You could mention, for example, what exactly do you plan to do in this project, how, the timeline. A bit about yourself. And so on.

Regards, Sergei

Hi Sriram. If you could add "CREATE VIEW … FORCE" too, that would be awesome. See the Oracle Enterprise docs for the behavior: http://docs.oracle.com/cd/B19306_01/server.102/b14200/statements_8004.htm Thanks, James. -------------------------------------------- On Wed, 3/19/14, Sergei Golubchik wrote: Subject: Re: [Maria-developers] [GSOC 2014] "CREATE OR REPLACE, CREATE IF NOT EXISTS, and DROP IF EXISTS" Project To: "sriram patil" Cc: maria-developers@lists.launchpad.net Date: Wednesday, March 19, 2014, 12:22 PM Hi, sriram! On Mar 19, sriram patil wrote:

Hi Sergei,

I studied lex and yacc and understood complete parser written in sql_yacc.yy. Also went through the constants and symbols declared in lex.h, sql_yacc.h, sql_cmd.h.

Right

I found out that OR REPLACE and IF NOT EXISTS is already implemented for tables. The IF NOT EXISTS rule is also defined for index in yacc file but not handled in the execution code. And for other object types we need to add the rules in yacc files and also need to change the code in "mysql_create_" functions in respective files.

Right

The OR REPLACE and IF NOT EXISTS are handled in the create execution itself. I found two flags in LEX class which are used for the purpose, create_info.options and check_exists.

So, are all the things that I have compiled correct?

Yes

I have an initial draft of my proposal ready. I will upload it on the GSOC portal today, please review. Should I also email the same to you?

No, please don't. I get an email automatically for every new proposal. Regards, Sergei _______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to     : maria-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~maria-developers More help   : https://help.launchpad.net/ListHelp

Hi James, Replying again cause the text did not appear in the mailing list. Sure, I will try and implement that feature too. Thank you very much for the suggestion. Thanks, Sriram On Thu, Mar 20, 2014 at 4:02 AM, James Briggs wrote:

Hi Sriram.

If you could add "CREATE VIEW ... FORCE" too, that would be awesome.

See the Oracle Enterprise docs for the behavior: http://docs.oracle.com/cd/B19306_01/server.102/b14200/statements_8004.htm

Thanks, James.

-------------------------------------------- On Wed, 3/19/14, Sergei Golubchik wrote:

Subject: Re: [Maria-developers] [GSOC 2014] "CREATE OR REPLACE, CREATE IF NOT EXISTS, and DROP IF EXISTS" Project To: "sriram patil" Cc: maria-developers@lists.launchpad.net Date: Wednesday, March 19, 2014, 12:22 PM

Hi, sriram!

On Mar 19, sriram patil wrote:

...

Hi Sergei,

I studied lex and yacc and understood complete parser written in sql_yacc.yy. Also went through the constants and symbols declared in lex.h, sql_yacc.h, sql_cmd.h.

Right

...

I found out that OR REPLACE and IF NOT EXISTS is already implemented for tables. The IF NOT EXISTS rule is also defined for index in yacc file but not handled in the execution code. And for other object types we need to add the rules in yacc files and also need to change the code in "mysql_create_" functions in respective files.

Right

...

The OR REPLACE and IF NOT EXISTS are handled in the create execution itself. I found two flags in LEX class which are used for the purpose, create_info.options and check_exists.

So, are all the things that I have compiled correct?

Yes

...

I have an initial draft of my proposal ready. I will upload it on the GSOC portal today, please review. Should I also email the same to you?

No, please don't. I get an email automatically for every new proposal.

Regards, Sergei

_______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to : maria-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~maria-developers More help : https://help.launchpad.net/ListHelp

Hi James, That worklog surely helped me. Now I know what needs to be added for each object type. I have missed three of them while putting up my proposal. Also, it seems the CREATE OR REPLACE and IF [NOT] EXISTS issue is hanging there from a long time. The follow-up project idea is great! That huge switch-case statement in mysql_command_execute can be fixed with the mentioned object oriented approach for command execution. And that blog of yours really simplifies it all. I will need to go through that procedure a lot of times during this project. I am so much excited now after I saw the complete worklog and so many things to work on. Thanks a lot, Sriram On Fri, Mar 21, 2014 at 3:45 AM, James Briggs wrote:

Hi Sriram.

1) Here's a worklog related to your project: WL#3129: Consistent clauses in CREATE and DROP https://dev.mysql.com/worklog/

2) Maybe you can do a follow-on GSOC next year on this worklog: WL#5047: SQL command class hierarchy.

There's lots of low-hanging fruit enhancing SQL options, yet at the same time very useful to end-users.

3) You can see my notes and references for adding new MySQL commands here:

Howto Add a New Command to the MySQL Server

http://www.jebriggs.com/blog/2013/05/howto-add-a-new-command-to-the-mysql-se...

James.

-------------------------------------------- On Thu, 3/20/14, sriram patil wrote:

Subject: Re: [Maria-developers] [GSOC 2014] "CREATE OR REPLACE, CREATE IF NOT EXISTS, and DROP IF EXISTS" Project To: "James Briggs" Cc: maria-developers@lists.launchpad.net Date: Thursday, March 20, 2014, 5:34 AM

Hi James, Replying again cause the text did not appear in the mailing list. Sure, I will try and implement that feature too. Thank you very much for the suggestion.

Thanks, Sriram

On Thu, Mar 20, 2014 at 4:02 AM, James Briggs wrote:

Hi Sriram.

If you could add "CREATE VIEW ... FORCE" too, that would be awesome.

See the Oracle Enterprise docs for the behavior:

http://docs.oracle.com/cd/B19306_01/server.102/b14200/statements_8004.htm

Thanks, James.