summaryrefslogtreecommitdiffstats
path: root/filedistribution
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2021-06-22 08:58:32 +0200
committerHarald Musum <musum@verizonmedia.com>2021-06-22 08:58:32 +0200
commit27208993ceebf45b2df9c7546d660af0b90cad9f (patch)
tree1f80dae57eac7dbda0f72c5ceaea59dbc11e4113 /filedistribution
parent8588e7ccf1f2b747bf3b5ac12a786dba3c330dde (diff)
Always setr download status when downloading completed
Diffstat (limited to 'filedistribution')
-rw-r--r--filedistribution/src/main/java/com/yahoo/vespa/filedistribution/Downloads.java2
-rw-r--r--filedistribution/src/test/java/com/yahoo/vespa/filedistribution/FileDownloaderTest.java1
2 files changed, 1 insertions, 2 deletions
diff --git a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/Downloads.java b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/Downloads.java
index d364e9ec48d..181c424ef98 100644
--- a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/Downloads.java
+++ b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/Downloads.java
@@ -38,8 +38,8 @@ public class Downloads {
void completedDownloading(FileReference fileReference, File file) {
Optional<FileReferenceDownload> download = get(fileReference);
+ setDownloadStatus(fileReference, 1.0);
if (download.isPresent()) {
- downloadStatuses().get(fileReference).ifPresent(Downloads.DownloadStatus::finished);
downloads.remove(fileReference);
download.get().future().complete(Optional.of(file));
} else {
diff --git a/filedistribution/src/test/java/com/yahoo/vespa/filedistribution/FileDownloaderTest.java b/filedistribution/src/test/java/com/yahoo/vespa/filedistribution/FileDownloaderTest.java
index 287cc7a74f3..475228543eb 100644
--- a/filedistribution/src/test/java/com/yahoo/vespa/filedistribution/FileDownloaderTest.java
+++ b/filedistribution/src/test/java/com/yahoo/vespa/filedistribution/FileDownloaderTest.java
@@ -122,7 +122,6 @@ public class FileDownloaderTest {
// Verify download status when downloaded
System.out.println(downloads.downloadStatuses());
- double downloadStatus = downloads.downloadStatus(fileReference);
assertDownloadStatus(fileReference, 1.0);
}