maria-developers team mailing list archive
Mailing list archive
Re: [GSoC] First two weeks into coding! - James' comments
I read your CREATE OR REPLACE DATABASE
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?
On Sun, 6/1/14, sriram patil <spsrirampatil@xxxxxxxxx> wrote:
Subject: [Maria-developers] [GSoC] First two weeks into coding!
Date: Sunday, June 1, 2014, 8:15 PM
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