DataUpdate::runUpdates() object handle transations automatically,
code in doUpdate should not interfere with that.
Change-Id: I7575e1b77d58921813af24e015edb03cad5fdb3a
/**
* Abstract base class for update jobs that do something with some secondary
* data extracted from article.
+ *
+ * @note: subclasses should NOT start or commit transactions in their doUpdate() method,
+ * a transaction will automatically be wrapped around the update. If need be,
+ * subclasses can override the beginTransaction() and commitTransaction() methods.
*/
abstract class DataUpdate implements DeferrableUpdate {
/**
* Abstract base class for update jobs that put some secondary data extracted
* from article content into the database.
+ *
+ * @note: subclasses should NOT start or commit transactions in their doUpdate() method,
+ * a transaction will automatically be wrapped around the update. Starting another
+ * one would break the outer transaction bracket. If need be, subclasses can override
+ * the beginTransaction() and commitTransaction() methods.
*/
abstract class SqlDataUpdate extends DataUpdate {