Chuck Bell wrote:
>> 2. I do not understand why the binlog handling is necessary in
>> Please, explain. See  below.
> We must ensure the entries written to the backup logs are not replicated
> to the slave. They should never be replicated. See R06.
But can't you turn it off for the entire backup/restore? Do you have to
turn it off for every write?
>> 3. Please, verify that restore may not enable slave connections after
>> restore when it was disabled before restore. 
> This is ok. We set disable_slaves = FALSE regardless of whether we
> turned it on (set to TRUE) or not. So this is safe to call without a
> guard or checking to see if it was set to TRUE previously.
And there can not be a scenario where a user has disabled slaves prior
to restore and do not want to have them enabled as a side effect of
>> 4. Please, use SET SESSION debug="-d" to turn off set_backup_id.
>> Otherwise, test will generate 20MB of master.err which takes a very
>> long time to parse at the end of the test. 
> Done. This must be a new and recent change to MTR or something because
> it never caused this problem before. Can you please check other backup
> tests and if you find them in those tests issue a bug report to fix
> them? Thanks.