← Back to team overview

maria-developers team mailing list archive

Re: [GSoC] First two weeks into coding! - James' comments


Hi Sriram.

code and have some comments:

1) I noticed that you directly mysql_rm_db the database
before creating it. Please ask your mentor if
it would be better to rename it, obtain a 
successful create status, then delete the old db.

Currently, if the disk is full, you could lose
the original db and also the new one.

2) mysql_rm_db has a side-effect of USE nothing;
if the db name matches the current db.

You may want to do mysql_change_db_impl()
 after the create logic to undo that.

3) This code impacts binary logging/replication, so you
need to test that as well.

4) Old comments refer to automatic DDL locking.
You should trace how that works (or not)
with your code. For example, are you 100% sure
another thread cannot create "db" after
the mysql_rm_db call?

" These DDL methods and logging are protected with the exclusive
metadata lock on the schema." Verify?

5) I like your blog effort. Can you add links to
github files so it's easier for reviewers to navigate?

Thanks, James.

On Sun, 6/1/14, sriram patil <spsrirampatil@xxxxxxxxx> wrote:

 Subject: [Maria-developers] [GSoC] First two weeks into coding!
 To: maria-developers@xxxxxxxxxxxxxxxxxxx
 Date: Sunday, June 1, 2014, 8:15 PM
 Hi all,
 I have created a blog to write the details of the
 work that I do as part of GSoC'14. Following is the link
 to my first blog post explaining the details of the work
 that I have done during the last two weeks.
 I will be writing these blog posts every weekend
 from now on and will share it on the developer mailing list
 every Monday. Suggestions/reviews are welcome.
 -----Inline Attachment Follows-----
 Mailing list: https://launchpad.net/~maria-developers
 Post to     : maria-developers@xxxxxxxxxxxxxxxxxxx
 Unsubscribe : https://launchpad.net/~maria-developers
 More help   : https://help.launchpad.net/ListHelp

Follow ups