OpenTreeOfLife / taxomachine

taxonomy graphdb
Other
7 stars 4 forks source link

Merge recent taxomachine changes to production (Cavia) #58

Closed jimallman closed 10 years ago

jimallman commented 10 years ago

This looks safe to me. Offhand, I don't see any major changes to the underlying graph database or its structure. A quick test (pushing master branch to production) also seems to be working properly, for both the synth-tree viewer and the curation app.

chinchliff commented 10 years ago

It should be safe, it's been reasonably well tested (given our testing standards) and includes a number of bug fixes. There were a couple of changes to the graph structure (different property names), but I think they have already been reflected on production, since I believe Jonathan recently pushed the new taxomachine db from devapi over to production, and it included those changes (@jar398, can you confirm?). Anyway @jimallman, if you have tested the services with your curator tools and the tree browser, then we should be good. I don't think there are any changes that would affect other clients. If there are problems, everything has been working ok on devap/development, so we should be able to migrate components over (like the taxomachine db, which will likely need to happen it hasn't already) if necessary.

On Wednesday, August 20, 2014, Jim Allman notifications@github.com wrote:

Assigned #58 https://github.com/OpenTreeOfLife/taxomachine/pull/58 to @chinchliff https://github.com/chinchliff.

— Reply to this email directly or view it on GitHub https://github.com/OpenTreeOfLife/taxomachine/pull/58#event-155355165.

jar398 commented 10 years ago

Cody, the latest taxomachine database upload to production was July 23. There was a taxomachine plugin checkout on production on July 24 (96e8) and then Jim just now checked our 3651. I'm not sure what graph structure changes you are referring to or what you mean by "recently" but if the changes were in 96e8 then we're good.

chinchliff commented 10 years ago

Yes, that should be fine then. The changes to the properties and associated services were made in 96e8.