review

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/SnapshotDataStoreDao.java

@@ -111,5 +111,17 @@ StateDao<ObjectInDataStoreStateMachine.State, ObjectInDataStoreStateMachine.Even
 
     int expungeBySnapshotList(List<Long> snapshotIds, Long batchSize);
 
+    /**
+     * Returns the total physical size, in bytes, of all snapshots stored on primary
+     * storage for the specified account that have not yet been backed up to
+     * secondary storage.
+     *
+     * <p>If no such snapshots are found, this method returns {@code 0}.</p>
+     *
+     * @param accountId the ID of the account whose snapshots on primary storage
+     *                  should be considered
+     * @return the total physical size in bytes of matching snapshots on primary
+     *         storage, or {@code 0} if none are found
+     */
     long getSnapshotsPhysicalSizeOnPrimaryStorageByAccountId(long accountId);
 }

engine/schema/src/main/java/org/apache/cloudstack/storage/datastore/db/SnapshotDataStoreDaoImpl.java

@@ -587,18 +587,6 @@ public class SnapshotDataStoreDaoImpl extends GenericDaoBase<SnapshotDataStoreVO
         return batchExpunge(sc, batchSize);
     }
 
-    /**
-     * Returns the total physical size, in bytes, of all snapshots stored on primary
-     * storage for the specified account that have not yet been backed up to
-     * secondary storage.
-     *
-     * <p>If no such snapshots are found, this method returns {@code 0}.</p>
-     *
-     * @param accountId the ID of the account whose snapshots on primary storage
-     *                  should be considered
-     * @return the total physical size in bytes of matching snapshots on primary
-     *         storage, or {@code 0} if none are found
-     */
     @Override
     public long getSnapshotsPhysicalSizeOnPrimaryStorageByAccountId(long accountId) {
         long snapshotsPhysicalSize = 0;