Przeglądaj źródła

Merge pull request #12368 from overleaf/jlm-jpa-switch-to-json-parse-from-bytes

[docstore] potential optimization for parsing a large JSON from buffer

GitOrigin-RevId: 1c2fb82eb27e2f5d58f75b99fab5dffc2d9b51d0
Jakob Ackermann 3 lat temu
rodzic
commit
24d2cf426f
1 zmienionych plików z 3 dodań i 4 usunięć
  1. 3 4
      services/docstore/app/js/DocArchiveManager.js

+ 3 - 4
services/docstore/app/js/DocArchiveManager.js

@@ -136,8 +136,7 @@ async function getDoc(projectId, docId) {
     })
   }
 
-  const json = buffer.toString()
-  return _deserializeArchivedDoc(json)
+  return _deserializeArchivedDoc(buffer)
 }
 
 // get the doc and unarchive it to mongo
@@ -199,8 +198,8 @@ async function _streamToBuffer(projectId, docId, stream) {
   })
 }
 
-function _deserializeArchivedDoc(json) {
-  const doc = JSON.parse(json)
+function _deserializeArchivedDoc(buffer) {
+  const doc = JSON.parse(buffer)
 
   const result = {}
   if (doc.schema_v === 1 && doc.lines != null) {