maria-developers team mailing list archive
-
maria-developers team
-
Mailing list archive
-
Message #09101
Re: [Commits] 9d8ad0d: Add --persistent option for mysqlcheck
Git log show his name, I think that the commit message generated by the
hook is at fault.
On Tue, 15 Dec 2015 at 16:20 Vicențiu Ciorbaru <cvicentiu@xxxxxxxxx> wrote:
> Hi Sergey,
>
> The commit message still has his name on it.
>
> I've only added some code to it, but I've left his original author.
>
> Vicentiu
>
> On Tue, 15 Dec 2015 at 11:32 Sergey Vojtovich <svoj@xxxxxxxxxxx> wrote:
>
>> Hi Vicentu,
>>
>> this is contribution by Daniel. He should get some credit for this.
>> Assuming you
>> didn't change this code much, this should be committed on Daniel's
>> behalf, so
>> that he gets credit on github. Or at the very least mention in a comment
>> contribution author.
>>
>> Thanks,
>> Sergey
>>
>> On Tue, Dec 15, 2015 at 11:11:50AM +0200, Vicentiu Ciorbaru wrote:
>> > revision-id: 9d8ad0d79e3c3a1a9bf7c8cc6477ee4e69fba633
>> (mariadb-10.0.22-63-g9d8ad0d)
>> > parent(s): e9b4a041af5122dffd59012493c565e6e3db2664
>> > committer: Vicențiu Ciorbaru
>> > timestamp: 2015-12-15 11:11:28 +0200
>> > message:
>> >
>> > Add --persistent option for mysqlcheck
>> >
>> > 10.0 has an "analyze table .. persistent for all" syntax. This adds
>> > --persistent to mysqlcheck(mysqlanalyize) to perform this extended
>> > analyze table option.
>> >
>> > ---
>> > client/mysqlcheck.c | 6 +++++-
>> > man/mysqlcheck.1 | 18 +++++++++++++++++-
>> > 2 files changed, 22 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/client/mysqlcheck.c b/client/mysqlcheck.c
>> > index ead4127..b9e5478 100644
>> > --- a/client/mysqlcheck.c
>> > +++ b/client/mysqlcheck.c
>> > @@ -43,7 +43,7 @@ static my_bool opt_alldbs = 0, opt_check_only_changed
>> = 0, opt_extended = 0,
>> > opt_silent = 0, opt_auto_repair = 0, ignore_errors = 0,
>> > tty_password= 0, opt_frm= 0, debug_info_flag= 0,
>> debug_check_flag= 0,
>> > opt_fix_table_names= 0, opt_fix_db_names= 0,
>> opt_upgrade= 0,
>> > - opt_do_tables= 1;
>> > + opt_persistent_all= 0, opt_do_tables= 1;
>> > static my_bool opt_write_binlog= 1, opt_flush_tables= 0;
>> > static uint verbose = 0, opt_mysql_port=0;
>> > static int my_end_arg;
>> > @@ -160,6 +160,9 @@ static struct my_option my_long_options[] =
>> > {"password", 'p',
>> > "Password to use when connecting to server. If password is not
>> given, it's solicited on the tty.",
>> > 0, 0, 0, GET_STR, OPT_ARG, 0, 0, 0, 0, 0, 0},
>> > + {"persistent", 'Z', "When using ANALYZE TABLE use the PERSISTENT FOR
>> ALL option.",
>> > + &opt_persistent_all, &opt_persistent_all, 0, GET_BOOL, NO_ARG, 0, 0,
>> > + 0, 0, 0, 0},
>> > #ifdef __WIN__
>> > {"pipe", 'W', "Use named pipes to connect to server.", 0, 0, 0,
>> GET_NO_ARG,
>> > NO_ARG, 0, 0, 0, 0, 0, 0},
>> > @@ -909,6 +912,7 @@ static int handle_request_for_tables(char *tables,
>> size_t length, my_bool view)
>> > case DO_ANALYZE:
>> > DBUG_ASSERT(!view);
>> > op= (opt_write_binlog) ? "ANALYZE" : "ANALYZE NO_WRITE_TO_BINLOG";
>> > + if (opt_persistent_all) end = strmov(end, " PERSISTENT FOR
>> ALL");
>> > break;
>> > case DO_OPTIMIZE:
>> > DBUG_ASSERT(!view);
>> > diff --git a/man/mysqlcheck.1 b/man/mysqlcheck.1
>> > index c175483..e05bbf8 100644
>> > --- a/man/mysqlcheck.1
>> > +++ b/man/mysqlcheck.1
>> > @@ -1,6 +1,6 @@
>> > '\" t
>> > .\"
>> > -.TH "\FBMYSQLCHECK\FR" "1" "04/08/2015" "MariaDB 10\&.0" "MariaDB
>> Database System"
>> > +.TH "\FBMYSQLCHECK\FR" "1" "27/12/2015" "MariaDB 10\&.0" "MariaDB
>> Database System"
>> > .\" -----------------------------------------------------------------
>> > .\" * set default formatting
>> > .\" -----------------------------------------------------------------
>> > @@ -677,6 +677,22 @@ Specifying a password on the command line should
>> be considered insecure\&. You c
>> > .sp -1
>> > .IP \(bu 2.3
>> > .\}
>> > +.\" mysqlcheck: persisent option
>> > +.\" persistent option: mysql
>> > +\fB\-\-persistent\fR,
>> > +\fB\-Z\fR
>> > +.sp
>> > +Used with ANALYZE TABLE to append the option PERSISENT FOR ALL.
>> > +.RE
>> > +.sp
>> > +.RS 4
>> > +.ie n \{\
>> > +\h'-04'\(bu\h'+03'\c
>> > +.\}
>> > +.el \{\
>> > +.sp -1
>> > +.IP \(bu 2.3
>> > +.\}
>> > .\" mysqlcheck: pipe option
>> > .\" pipe option: mysql
>> > \fB\-\-pipe\fR,
>> > _______________________________________________
>> > commits mailing list
>> > commits@xxxxxxxxxxx
>> > https://lists.askmonty.org/cgi-bin/mailman/listinfo/commits
>> _______________________________________________
>> commits mailing list
>> commits@xxxxxxxxxxx
>> https://lists.askmonty.org/cgi-bin/mailman/listinfo/commits
>
>
References