prune是坏的

[wellcoming]

失去了所有力气和手段
prune把我所有备份删了
我的配置:

{
  "enabled": true,
  "maxAmount": 0,
  "maxLifeTime": "0s",
  "last": 6,
  "hour": 12,
  "day": 7,
  "week": 4,
  "month": -1,
  "year": -1
}

之后我会提交一个PR已经提交PR #48

[wellcoming]

什么叫PBS遍历顺序是随机的

[wellcoming]

Map.of是哈希表Map，如果运行了错误的顺序会直接截断后面的槽位，所以每次prune会随机清空你的所有备份！！

[wellcoming]

气笑了

[wellcoming]

@SkyDynamic

[wellcoming]

顺序也错了，last应该先处理

[wellcoming]

QuickBackupM-Reforged/common/src/main/java/io/github/skydynamic/quickbackupmulti/schedule/runnables/DefaultPruneRunnable.java

Lines 54 to 118 in 190a425

	private static List<StorageInfo> filterBackupWithPbs(PbsConfig pbsConfig, List<StorageInfo> backupList, String timezoneOverride) {
	if (pbsConfig == null || !pbsConfig.isEnabled() || backupList == null || backupList.isEmpty()) {
	return new ArrayList<>();
	}
	ZoneId zoneId = timezoneOverride != null ? ZoneId.of(timezoneOverride) : ZoneId.systemDefault();
	List<StorageInfo> filteredList = new ArrayList<>(backupList);
	filteredList.sort(Comparator.comparingLong(StorageInfo::getTimestamp));
	List<StorageInfo> toDelete = new ArrayList<>();
	Map<String, Integer> timeUnits = Map.of(
	"hour", pbsConfig.getHour(),
	"day", pbsConfig.getDay(),
	"week", pbsConfig.getWeek(),
	"month", pbsConfig.getMonth(),
	"year", pbsConfig.getYear()
	);
	for (Map.Entry<String, Integer> entry : timeUnits.entrySet()) {
	String unit = entry.getKey();
	int count = entry.getValue();
	if (count <= 0) continue;
	Map<String, StorageInfo> latestPerUnit = new HashMap<>();
	for (StorageInfo backup : filteredList) {
	String key = DurationUtils.formatByUnit(backup.getTimestamp(), unit, zoneId);
	if (!latestPerUnit.containsKey(key)) {
	latestPerUnit.put(key, backup);
	} else if (backup.getTimestamp() > latestPerUnit.get(key).getTimestamp()) {
	toDelete.add(latestPerUnit.get(key));
	latestPerUnit.put(key, backup);
	} else {
	toDelete.add(backup);
	}
	}
	filteredList.removeAll(latestPerUnit.values());
	}
	if (pbsConfig.getLast() > 0 && filteredList.size() > pbsConfig.getLast()) {
	int keepCount = pbsConfig.getLast();
	List<StorageInfo> toKeep = filteredList.subList(filteredList.size() - keepCount, filteredList.size());
	Set<StorageInfo> toKeepSet = new HashSet<>(toKeep);
	toDelete.addAll(filteredList.stream().filter(b -> !toKeepSet.contains(b)).toList());
	}
	if (pbsConfig.getMaxLifeTime() != null && !pbsConfig.getMaxLifeTime().equals("0s")) {
	long maxLifeTimeMillis = DurationUtils.parseDurationToSeconds(pbsConfig.getMaxLifeTime()) * 1000;
	long now = System.currentTimeMillis();
	List<StorageInfo> expired = filteredList.stream()
	.filter(b -> now - b.getTimestamp() > maxLifeTimeMillis)
	.toList();
	toDelete.addAll(expired);
	filteredList.removeAll(expired);
	}
	if (pbsConfig.getMaxAmount() > 0 && filteredList.size() > pbsConfig.getMaxAmount()) {
	int removeCount = filteredList.size() - pbsConfig.getMaxAmount();
	toDelete.addAll(filteredList.subList(0, removeCount));
	}
	return toDelete;
	}

原prune存在多个互相影响的bug，接下来将一一分析

last规则居然在最后处理

keep-last : Keep the last backup snapshots.

由于last规则错误的置于处理后部，在处理了hour, month等众多规则后，last只能保留剩余的最后的几个槽位。根本无法保留最新快照。

被忽略的保存限制

count被读取后就没有使用，根本没有限制快照的保存数量，只是单纯的每个规则保存无限组。

随机的执行顺序

之前的几个还是无伤大雅，只要设置的没有触发bug还是能剩下一点数据的。
此bug是由于Map.of创建了一个哈希表，但由于哈希函数加盐的随机性，每次prune都是一次抽奖。如果抽到的顺序是正确的从小到大依次执行，那么你的数据会被惊险的保存下来。但如果有一个反向执行，那么它会将剩余的所有槽位加入删除列表，接下来所有备份都会被删除。而数据被正确处理的概率仅有0.8%（备份没丢失的建议去玩抽卡游戏）。

其他一些bug

还有几个bug。

• 临时备份无法使用。
• 如果服务器重启，备份/prune的计时器会重置，这代表如果设置一天一备份，服务器不连续运行一天根本不备份。
• 自动重启不兼容许多模组如Xaero世界地图
• qbm.accesswidener并没有在各子项目同步，导致非法访问

还有一些缺失的feature

• prune无法设置豁免
• qb list应默认显示最新快照而不是最老的
• qb purne指令用来手动清理与调试

[SkyDynamic]

有几点是刻意的：

• 临时备份本身就是用于处理回档异常的情况使用的
• 如果你设置的间隔过长，应使用crontab表达式而不是设置interval

关于别的点会在近期修复

[SkyDynamic]

该Issues关闭，对于其他feature你可以提交新的Issues并打上Feature标签