Skip to content

Commit 9dc1937

Browse files
authored
fix storage cleanup deleting the newest file instead of the oldest (#384)
1 parent 2bd6c30 commit 9dc1937

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

server/src/main/java/org/eclipse/jifa/server/repository/FileRepo.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,5 @@ public interface FileRepo extends CrudRepository<FileEntity, Long> {
2929

3030
Optional<FileEntity> findByUniqueName(String uniqueName);
3131

32-
Optional<FileEntity> findFirstByOrderByCreatedTimeDesc();
32+
Optional<FileEntity> findFirstByOrderByCreatedTimeAsc();
3333
}

server/src/main/java/org/eclipse/jifa/server/service/impl/FileServiceImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@ public NamedResource handleDownloadRequest(long fileId) throws Throwable {
304304

305305
@Override
306306
public void deleteOldestFile() {
307-
Optional<FileEntity> optional = fileRepo.findFirstByOrderByCreatedTimeDesc();
307+
Optional<FileEntity> optional = fileRepo.findFirstByOrderByCreatedTimeAsc();
308308
if (optional.isPresent()) {
309309
FileEntity file = optional.get();
310310
doDelete(file);

0 commit comments

Comments
 (0)