perf(server): don't create temp file on each file download

actorapp · Jan 6, 2016 · 8700df6 · 8700df6
1 parent 65ea511
commit 8700df6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/...actor-fs-adapters/src/main/scala/im/actor/server/file/local/LocalFileStorageAdapter.scala b/...actor-fs-adapters/src/main/scala/im/actor/server/file/local/LocalFileStorageAdapter.scala
@@ -118,9 +118,7 @@ final class LocalFileStorageAdapter(_system: ActorSystem)
     persist.FileRepo.find(id) flatMap {
       case Some(file) ⇒ DBIO.from(for {
         file ← getFile(file.id, file.name)
-        tmpDir ← Future(File.newTempDir())
-        copy ← Future(file.copyTo(tmpDir / file.name))
-      } yield Some(copy.toJava))
+      } yield Some(file.toJava))
       case None ⇒ DBIO.successful(None)
     }
 

diff --git a/actor-server/notes/1.0.85.markdown b/actor-server/notes/1.0.85.markdown
@@ -2,6 +2,7 @@ Bug fixes
 ===
 
 * Reduce GC overhead caused by Http Routes controlled by hooks
+* Don't create temp file on each file download
 
 Packaging
 ===
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,7 @@ Bug fixes @@
     ===
     * Reduce GC overhead caused by Http Routes controlled by hooks
+    * Don't create temp file on each file download
     Packaging
     ===
@@ Expand Down @@