I found another bug that was caused by this complex .equals, so I changed it to essentially "if uuid matches or internal id matches, return true" https://tickets.openmrs.org/browse/SYNC-231
Ben On Tue, Sep 20, 2011 at 8:49 PM, Maros Cunderlik <[email protected]>wrote: > For what it is worth, I don't remember any specific reason for this > .equal() being so complex. Maros > > > On Tue, Sep 20, 2011 at 5:06 AM, Ben Wolfe <[email protected]> wrote: > >> Looks like this is caused by the RemoteServer.equals method comparing >> ALL fields in RemoteServer instead of just uuids or ids. Anyone know >> why that is that way? >> >> I added a workaround to just look at ids instead of using .equals in >> trunk of sync. I'm not sure how long this has been there, but if its >> been there for a while your traffic back to child servers should drop >> by quite a bit! >> >> Ben >> >> On Mon, Sep 19, 2011 at 5:43 PM, Mark Goodrich <[email protected]> wrote: >> > Fyi, a sync bug I noticed on Friday. Not necessarily a blocker, but >> something we probably want to fix before releasing a version 1.0 >> > >> > https://tickets.openmrs.org/browse/SYNC-227 >> > >> > Mark >> > >> > _________________________________________ >> > >> > To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to >> [email protected] with "SIGNOFF openmrs-devel-l" in the body >> (not the subject) of your e-mail. >> > >> > [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l] >> > >> >> _________________________________________ >> >> To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to >> [email protected] with "SIGNOFF openmrs-devel-l" in the body >> (not the subject) of your e-mail. >> >> [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l] >> > > ------------------------------ > Click here to > unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from > OpenMRS Developers' mailing list _________________________________________ To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to [email protected] with "SIGNOFF openmrs-devel-l" in the body (not the subject) of your e-mail. [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]
