Closed
Bug 180870
Opened 22 years ago
Closed 22 years ago
Remove old shadowdb manual replication code
Categories
(Bugzilla :: Bugzilla-General, defect, P2)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: bbaetz, Assigned: bbaetz)
References
Details
Attachments
(1 file, 1 obsolete file)
18.77 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
Now that bug 124589 is in, we can pull out the old hacked up code for manual
replication.
Assignee | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Summary: Remove old shadowdb manual replicaiton code → Remove old shadowdb manual replication code
Target Milestone: --- → Bugzilla 2.18
Assignee | ||
Comment 1•22 years ago
|
||
OK, this removes the support code for syncshadowdb.
The only issue is that people using a shadowdb who are upgrading will end up
failing because the host/port will be set to ''. Results would be incorrect in
any case, though, due to the lack of syncshadowdb, so I don't think we should
care about that.
This has been lightly tested locally, and I don't intend to commit this until
after bmo uses replication.
Assignee | ||
Updated•22 years ago
|
Attachment #106775 -
Flags: review?(myk)
Assignee | ||
Comment 2•22 years ago
|
||
The patch will have a small conflict in defparams.pl wrt a description change;
should be trival to manually fix for a review.
Assignee | ||
Comment 3•22 years ago
|
||
Fixes cvs conflicts, and also removes the lcok code in post_bug, which isn't
needed since mysql handles auto_incrememnt replicaiton properly.
Attachment #106775 -
Attachment is obsolete: true
Assignee | ||
Updated•22 years ago
|
Attachment #106775 -
Flags: review?(myk)
Assignee | ||
Comment 4•22 years ago
|
||
Comment on attachment 107325 [details] [diff] [review]
v2
This is basically jsut code removal, and should be trivial once replication is
tested on bmo
Attachment #107325 -
Flags: review?(myk)
Assignee | ||
Comment 5•22 years ago
|
||
Comment on attachment 107325 [details] [diff] [review]
v2
Note that removing the synshadowdb script is also part of this patch
Attachment #107325 -
Flags: review?(bugreport)
Comment 6•22 years ago
|
||
I'm confused about this patch. It seems to be leaving some replication functions
in place. How much is supposed to be going and how much is supposed to stay?
Assignee | ||
Comment 7•22 years ago
|
||
Whats gone is the bit where we do the replicaiton (ie syncshadowdb) We still
support a separate database to work arround mysql's table-level locking, but w
enow require something else to handle moving data from one table to the other.
Comment 8•22 years ago
|
||
Comment on attachment 107325 [details] [diff] [review]
v2
r=myk
Attachment #107325 -
Flags: review?(myk) → review+
Updated•22 years ago
|
Flags: approval+
Assignee | ||
Comment 9•22 years ago
|
||
Checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•22 years ago
|
Attachment #107325 -
Flags: review?(bugreport)
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•