Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1272#issuecomment-172323097
@koushik-das your change makes perfect sense to me from a db housekeeping
point of view. I have one (little) doubt; what are the semantic ramifications.
An asynchronous job might have continued after MS shutdown and its result might
be actually success or failure. I am not sure if diving into this field is
appropriate for this PR. Just checking with you as you might have already
considered it.
code LGTM
---
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.
---
