Calling `issueFullFileCheck()` does not seem to remove old unreferenced objects

[andriikovalov]

Calling `issueFullFileCheck()` does not seem to remove old unreferenced objects

[hg-ms]

Hello,
This not a bug. In your example the storage has no time to clean up old data.
The point of time when data gets deleted depends on several factors, the most important ones are:

• The Java GC
• The storage GC
• The storage GC configuration
• The storage internal cache
• The write load.
  In your example you create a heavy write load that requires some tweaking of the storage housekeeping.

The Example below should perform better regarding the cleanup.
It ensures that the Java GC and storage GC are executed and sets a very small object live time for the storage cache and increases the time budget for housekeeping.

final EmbeddedStorageManager storageManager = EmbeddedStorage
	.start(Storage.ConfigurationBuilder()
		.setEntityCacheEvaluator(Storage.EntityCacheEvaluator(1000, 10))
		.setHousekeepingController(Storage.HousekeepingController(100, 1000_000_000))
		.setStorageFileProvider(Storage.FileProvider(storagePath)).createConfiguration());

for (int i = 0; i < 50; i++) {
	storageManager.setRoot(new byte[1000000]); // 1 MB
	storageManager.storeRoot();
}

System.gc();
storageManager.issueFullGarbageCollection();
storageManager.issueFullFileCheck();

[andriikovalov]

Okay, thank you. I thought that explicitly calling housekeeping would delete unreachable data (as mentioned in #179), but in fact it is not guaranteed, and cannot be enforced, right?
Your snippet with "aggressive housekeeping" also gives me the same result (storage not shrinked).