Github user abhinandanprateek commented on the pull request:
https://github.com/apache/cloudstack/pull/689#issuecomment-134480002
I think we should not be using TransactionLegacy anymore. It was used in
first place just to be consistent with UsageServer and due to time constraints.
I think if we should revamp this by using the new translation framework and do
it like:
Transaction.execute(new TransactionCallback<Object>() {
@Override
public Object doInTransaction(TransactionStatus status) {
// ... do something ...
return myresult;
}
});
This is described here by Darren
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Database+Transactions.
I propose that we revamp the Quota with the new Transaction model and as
well as the usage server as just the one without the other is still half work.
Will work on that.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
