While we're at it, can we change cohorts to use uuids rather than patient ids too? On Sep 16, 2011 5:56 PM, "Friedman, Roger (CDC/CGH/DGHA) (CTR)" < [email protected]> wrote: > The original proposal was to replace the autogenerated id with the UUID as the Hibernate identity field. Was that not implemented? > > Some of the linking tables had no ID or UUID, using the from/to unique ids as the unique id (between location and location tag, if I remember correctly, is one). Has that been changed? > > From: [email protected] [mailto:[email protected]] On Behalf Of Darius Jazayeri > Sent: Friday, September 16, 2011 11:20 AM > To: [email protected] > Subject: Re: [OPENMRS-DEV] TRUNK-2362 - Get rid of implementations of equals and hashCode in our domain objects > > > Roger, > > This only affects equality and hash set membership tests in java. It will be dwarfed by the cost of pulling things from the database, from a performance standpoint. > > -Darius (by phone) > On Sep 16, 2011 7:14 AM, "Friedman, Roger (CDC/CGH/DGHA) (CTR)" < [email protected]<mailto:[email protected]>> wrote: > Greg -- > You don't have to break everything to be famous, all you have to do is slow things down a lot. The article that is at the root of this issue rejected UUIDs because comparisons are slow due to their length. Could you try some operations with your fix and the large DB and compare performance? > Saludos, Roger > > From: [email protected]<mailto:[email protected]> [mailto:[email protected] <mailto:[email protected]>] On Behalf Of Greg Warren > Sent: Friday, September 16, 2011 1:06 AM > To: [email protected]<mailto: [email protected]> > Subject: [OPENMRS-DEV] TRUNK-2362 - Get rid of implementations of equals and hashCode in our domain objects > > > > > I am going to check in this fix which affects almost all the domain objects. We have removed t... > > > ________________________________ > Click here to unsubscribe from OpenMRS Developers' mailing list > > ________________________________ > Click here to > unsubscribe<mailto:[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]
_________________________________________ 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]

