Re: [Xtrabackup Manager] discussion

Skip to first unread message

Lachlan Mulcahy

Mar 28, 2011, 12:37:25 PM3/28/11
Hi Henrik,

To avoid any confusion that Percona are developing Xtrabackup Manager (which could easily happen due to the naming), I created a new Google Group called "xtrabackup-mgr".

Cross posting this for now, but let's continue discussion there!



On 28/03/2011, at 8:03 AM, Henrik Ingo wrote:

> Hi Lachlan
> So the final ingredient in running a successful open source project:
> avoid private emails, push everything to a mailing list or IRC.
> I'll make a bold assumption that the Percona guys don't mind if we
> discuss Xtrabackup Manager on this list that is also home for
> Xtrabackup development.
> Here continues random review'ish notes from today's experiments:
> **
> Commit message for r16:
> Also disabled PHP warnings in config.php by default for cleaner output.
> This bit me a little. By default Ubuntu comes without php-mysql
> installed, so anything in xbm fails due to missing the mysqli library.
> The first time this fails is just subclassing the dbConnection from
> mysqli and since no object or even the class itself doesn't exist at
> this point, of course your error handling doesn't do anything either.
> So the result is that the script silently fails and does nothing.
> I'd personally let the default be to PHP print the errors as long as
> heavy development goes on.
> **
> You are being very careful with validating input for correctness, etc.
> Unfortunately this makes the code verbose and hard to read. Going
> forward it would be great if you adopted the use of exceptions for
> signaling errors, so that one needn't check the return value of every
> single function call.
> With exceptions you can run through a large block of code without
> worrying about errors, then add a catch() statement at the end. This
> makes code much more readable. (And hey, this is PHP. Many of us
> writes PHP code without worrying about errors altogether :-)
> The good thing about exceptions is that you can also throw them from a
> object constructor. For instance, in backupSnapshot.class.php this
> check:
> if(!is_numeric($this->id)) {
> $this->error = 'backupSnapshot->setSnapshotTime:
> '."Error: The ID for this object is not an integer.";
> return false;
> }
> Should be done in the constructor. (But since you cannot return false
> from a constructor, I understand why you didn't. Use exceptions!)
> Ok, done for today.
> henrik
> --
> +358-40-5697354 skype: henrik.ingo irc: hingo
> My LinkedIn profile:

Reply all
Reply to author
0 new messages