Code review comment for lp:~stub/launchpad/auth-mirror-tables
Revision history for this message
| Guilherme Salgado (salgado) wrote | # |
review:
Approve
(release-critical)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
This looks fine but I'm not really fond of maintaining a copy of the list of columns in the Person table in the update trigger. I guess there isn't much we can do about it, though?
Also, we need to test this manually on staging as I guess there won't be enough time to have it rolled out as part of the next staging update.