On Mon, Feb 27, 2012 at 8:01 AM, Brian Aker <[email protected]> wrote: > On Feb 18, 2012, at 4:13 AM, David Shrewsbury wrote: >> transaction_log - Should not be enabled by default. This is >> the file-based transaction log (not used by the slave) and >> is rather buggy, IMO. I almost feel like we should remove it >> from the tree... > > I tend to agree,.... > > Unless someone else cares to do it, I can submit this patch.
As your new all-round packager, I'm not supportive of removing this for 7.1 tree. I do support the idea if it is implemented in the future 7.2 tree. Removals like this disrupt the packaging process. Since we are in RC we should not consider such deletions anymore, otherwise things will never get stable. From my point of view it is easier to have the unused plugin hanging around for 7.1 and then delete it as soon as we open a 7.2 development branch. We missed the boat with 7.1, but once we get into 7.2 I hope we can merge the packaging branch into main tree finally. Then these kind of changes can be done with one patch that both deletes the code and makes necessary rpm .spec and debian/* changes, causing less side effects. henrik -- [email protected] +358-40-8211286 skype: henrik.ingo irc: hingo www.openlife.cc My LinkedIn profile: http://www.linkedin.com/profile/view?id=9522559 _______________________________________________ Mailing list: https://launchpad.net/~drizzle-discuss Post to : [email protected] Unsubscribe : https://launchpad.net/~drizzle-discuss More help : https://help.launchpad.net/ListHelp
