----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6325/ -----------------------------------------------------------
(Updated Aug. 11, 2012, 1:13 a.m.) Review request for cloudstack, Nitin Mehta and edison su. Changes ------- marked failed snapshots as Error and expunge them in a background thread, the cleanup interval is storage.cleanup.interval Summary (updated) ----------------- CS-15823 Failed snapshot should be marked as Error and cleanup asynchronously Description (updated) ------- changes: marked failed snapshots as Error and expunge them in a background thread, the cleanup interval is storage.cleanup.interval This addresses bug CS-15823. Diffs (updated) ----- server/src/com/cloud/storage/StorageManagerImpl.java c17379f server/src/com/cloud/storage/dao/SnapshotDao.java 4bf54de server/src/com/cloud/storage/dao/SnapshotDaoImpl.java ac4bb6f server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java 6e3f9c1 Diff: https://reviews.apache.org/r/6325/diff/ Testing ------- verified Thanks, mice xia
