Comment 1 for bug 223906

Revision history for this message
danh (danh-archive) wrote :

The current plan is to modify petabox/deriver/inc/Deriver.inc right after it calls the preDerive() method of the BookItemOperation class.

This is based on conversations with Steve, Raj, and Hank (although the exact details here may have to be modified yet more).

There will be a function call made right at that point to some function (which will wrap around the metaTableUpdater) to update the metadata.

Right now, the preDerive is wrapped in a try {} catch block.

The new call could go inside that block, or immediately afterwards.

My plan is to put it immediately afterwards, because the policy on whether to fail due to an unsuccessful book operation is not necessarily the same as the policy on whether to fail due to inability to write to the database.

Because access to the database is not guaranteed, and there will be a second chance to write to the database at the end of the derive (which might be days later), it would seem better to at least have the option of not failing on database access, but definitely failing on book operation malfunction.

Anybody reading this please correct me, and thanks in advance.