From: Sergey Vojtovich
Date: November 18 2010 2:41pm
Subject: Re: bzr commit into mysql-5.1-bugteam branch
 (nirbhay.choubey:3554) Bug#54899
List-Archive: http://lists.mysql.com/commits/124340
Message-Id: <20101118144122.GA8452@june>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

Hi Nirbhay,

I assume we agree that the problem with use case #5 requires
code change. I'm looking for next patch to continue objective
discussion.

Thank,
Sergey

On Thu, Nov 18, 2010 at 07:35:39PM +0530, Nirbhay Choubey wrote:
> Hi Sergey,
> 
> On Thursday 18 November 2010 06:19 PM, Sergey Vojtovich wrote:
> >Hi Nirbhay, Joro!
> >
> >Use case #5:
> >I have a dump provided by mysqldump tool:
> >--
> >   USE `non_existent_db`;
> >   CREATE TABLE `table_in_non_existent_db`(a INT);
> >   USE `test`;
> >   CREATE TABLE `table_in_test`(a INT);
> >--
> >Note: I simplified dump for better readability.
> >
> >I have no `non_existent_db` database.
> >I have empty `test` database.
> >I want to load tables from `test` database.
> >I read manual for one-database option:
> >--
> >   Ignore statements except those that occur while the default database
> >   is the one named on the command line. This option is rudimentary and
> >   should be used with care. Statement filtering is based only on USE
> >   statements.
> >--
> >I think this option does what I need.
> >I do "mysql --one-database test".
> >I expect new table in `test`: `table_in_test`.
> >Without this patch I get new table.
> >With this patch `test` database is empty.
> 
> This is a matter of choice of how we want mysql client to respond
> if it hits an erroneous statement while loading (with one-database option).
> 
> As 'post-patch' mysql client now throws an error while executing
> 'USE `non_existent_db`, it will skip the loading.
> This will happen with or without one-database option.
> 
> However, with pre-patch client,  'USE `non_existent_db` will just
> succeed, and hence, loading will continue.
> 
> If, we want the post-patch client to behave the same way, we can
> just change the error message to a *soft* error message to the user,
> and can let the loading proceed normally.
> 
> Thanks for pointing this out.
> 
> >Regards,
> >Sergey
> >
> >On Wed, Nov 17, 2010 at 10:09:46AM +0000, Nirbhay Choubey wrote:
> >>#At file:///home/nirbhay/Project/mysql/repo/wl/mysql-5.1-bugteam-54899/ based on revid:guilhem@stripped
> >>
> >>  3554 Nirbhay Choubey	2010-11-17
> >>             Bug#54899 : --one-database option cannot handle DROP/CREATE DATABASE commands
> >...skip...
> >
> >>@@ -1449,8 +1450,9 @@ static struct my_option my_long_options[
> >>     &opt_sigint_ignore,&opt_sigint_ignore, 0, GET_BOOL,
> >>     NO_ARG, 0, 0, 0, 0, 0, 0},
> >>    {"one-database", 'o',
> >>-   "Only update the default database. This is useful for skipping updates "
> >>-   "to other database in the update log.",
> >>+   "Ignore statements except those that occur while the default "
> >>+   "database is the one named at the command line and USE command "
> >>+   "is not executed with some other database.",
> >>     0, 0, 0, GET_NO_ARG, NO_ARG, 0, 0, 0, 0, 0, 0},
> >>  #ifdef USE_POPEN
> >>    {"pager", OPT_PAGER,
> >Are you sure your addendum is correct?
> 
> Yes, I tired to define it in the way client behaves as of now, but
> if we come up with a more precise definition of one-database,
> will modify it.
> 
> Best,
> Nirbhay
> 
> >Regards,
> >Sergey
> 

-- 
Sergey Vojtovich <svoj@stripped>
MySQL AB, Software Engineer
Izhevsk, Russia, www.mysql.com