Use of ModellerCore.startTxn do not always set a succeeded variable to true causing an erroneous transaction rollback

boolean started = ModelerCore.startTxn(false, false, "Some task", this);
boolean succeeded = false;

try {
  // do something useful here

  // This succeeded is not always set to true
  succeeded = true;

} catch (final Exception err) {

  // log the exception

} finally {
  if (started) {
    if (succeeded) {
      ModelerCore.commitTxn();
    } else {
      ModelerCore.rollbackTxn();
    }
  }
}