Code review comment for lp:~stewart/drizzle/slap-remove-autocommit-in-load-bug668143
Revision history for this message
| Brian Aker (brianaker) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Just looking at the patch, you only removed part of the feature, not the entire thing.
Looking at this, I think it might be better to just fix it to work, instead of pulling it out (which I know is counter to what I thought on IRC, but I hadn't looked at the code at that point).