diff --git a/src/main/java/org/qortal/controller/arbitrary/ArbitraryDataCleanupManager.java b/src/main/java/org/qortal/controller/arbitrary/ArbitraryDataCleanupManager.java index 7eaeb44c..4d349dce 100644 --- a/src/main/java/org/qortal/controller/arbitrary/ArbitraryDataCleanupManager.java +++ b/src/main/java/org/qortal/controller/arbitrary/ArbitraryDataCleanupManager.java @@ -4,10 +4,12 @@ import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; import org.qortal.api.resource.TransactionsResource.ConfirmationStatus; import org.qortal.data.transaction.ArbitraryTransactionData; +import org.qortal.data.transaction.TransactionData; import org.qortal.repository.DataException; import org.qortal.repository.Repository; import org.qortal.repository.RepositoryManager; import org.qortal.settings.Settings; +import org.qortal.transaction.Transaction; import org.qortal.transaction.Transaction.TransactionType; import org.qortal.utils.ArbitraryTransactionUtils; import org.qortal.utils.Base58; @@ -198,12 +200,26 @@ public class ArbitraryDataCleanupManager extends Thread { LOGGER.error("Repository issue when fetching arbitrary transaction data", e); } - // Delete additional data at random if we're over our storage limit - // Use a threshold of 1 so that we only start deleting once the hard limit is reached - // This also allows some headroom between the regular threshold (90%) and the hard - // limit, to avoid data getting into a fetch/delete loop. - if (!storageManager.isStorageSpaceAvailable(1.0f)) { - this.storageLimitReached(); + try (final Repository repository = RepositoryManager.getRepository()) { + + // Delete random data associated with name if we're over our storage limit for this name + // Use a threshold of 1 so that we only start deleting once the hard limit is reached + // This also allows some headroom between the regular threshold (90%) and the hard + // limit, to avoid data getting into a fetch/delete loop. + for (String followedName : storageManager.followedNames()) { + if (!storageManager.isStorageSpaceAvailableForName(repository, followedName, 1.0f)) { + this.storageLimitReachedForName(repository, followedName); + } + } + + // Delete additional data at random if we're over our storage limit + // Use a threshold of 1, for the same reasons as above + if (!storageManager.isStorageSpaceAvailable(1.0f)) { + this.storageLimitReached(repository); + } + + } catch (DataException e) { + LOGGER.error("Repository issue when cleaning up arbitrary transaction data", e); } } } catch (InterruptedException e) { @@ -211,7 +227,7 @@ public class ArbitraryDataCleanupManager extends Thread { } } - private void storageLimitReached() throws InterruptedException { + private void storageLimitReached(Repository repository) throws InterruptedException { // We think that the storage limit has been reached // Firstly, rate limit, to avoid repeated calls to calculateDirectorySize() @@ -230,19 +246,42 @@ public class ArbitraryDataCleanupManager extends Thread { // when they reach their storage limit Path dataPath = Paths.get(Settings.getInstance().getDataPath()); for (int i=0; i followedNames() { + return ResourceListManager.getInstance().getStringsInList("followed", "names"); + } + private int followedNamesCount() { return ResourceListManager.getInstance().getItemCountForList("followed", "names"); } diff --git a/src/test/java/org/qortal/test/common/ArbitraryUtils.java b/src/test/java/org/qortal/test/common/ArbitraryUtils.java new file mode 100644 index 00000000..aaadaa21 --- /dev/null +++ b/src/test/java/org/qortal/test/common/ArbitraryUtils.java @@ -0,0 +1,2 @@ +package org.qortal.test.common;public class ArbitraryUtils { +}