etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

guard GlobalLock lock with finally clause

This commit is contained in:
Kelven Yang 2011-08-04 16:18:45 -07:00
parent 41759c153d
commit d9392a6aa7
1 changed files with 22 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
server/src/com/cloud/domain/dao/DomainDaoImpl.java
View File

@ -105,26 +105,28 @@ public class DomainDaoImpl extends GenericDaoBase<DomainVO, Long> implements Dom
s_logger.error("Unable to lock parent domain: " + parent);
return null;
}
Transaction txn = Transaction.currentTxn();
try {
txn.start();
domain.setPath(allocPath(parentDomain, domain.getName()));
domain.setLevel(parentDomain.getLevel() + 1);
parentDomain.setNextChildSeq(parentDomain.getNextChildSeq() + 1); // FIXME: remove sequence number?
parentDomain.setChildCount(parentDomain.getChildCount() + 1);
persist(domain);
update(parentDomain.getId(), parentDomain);
txn.commit();
return domain;
} catch(Exception e) {
s_logger.error("Unable to create domain due to " + e.getMessage(), e);
txn.rollback();
return null;
} finally {
try {
Transaction txn = Transaction.currentTxn();
try {
txn.start();
domain.setPath(allocPath(parentDomain, domain.getName()));
domain.setLevel(parentDomain.getLevel() + 1);
parentDomain.setNextChildSeq(parentDomain.getNextChildSeq() + 1); // FIXME: remove sequence number?
parentDomain.setChildCount(parentDomain.getChildCount() + 1);
persist(domain);
update(parentDomain.getId(), parentDomain);
txn.commit();
return domain;
} catch(Exception e) {
s_logger.error("Unable to create domain due to " + e.getMessage(), e);
txn.rollback();
return null;
}
} finally {
lock.unlock();
}
}