fix parser options to workaround stream-json issue

Author: gregolsky

perf/results/load.md

@@ -148,37 +148,15 @@ async function enhancedStreamJson() {
 
 ## 4.0.4-load-full-pipeline 
 
-- x10: 1574.306ms
-- x50: 6226.822ms
-- x100: 12414.000ms
+- x10: 1784.831ms
+- x50: 7229.829ms
+- x100: 14543.986ms
 
 ```javascript
-async function enhancedStreamJson() {
+async function loadPipeline() {
     const dataStream = fs.createReadStream("./load_data.json");
-    const streams = [
-        dataStream,
-        parser({
-            packKeys: true,
-            packStrings: true,
-            packValues: true,
-            packNumbers: true,
-            streamNumbers: false,
-            streamValues: false,
-            streamKeys: false,
-            streamStrings: false
-        }),
-        new TransformKeysJsonStream({
-            getCurrentTransform: buildEntityKeysTransform("camel")
-        })
-    ];
-    const asm = Asm.connectTo(streams[streams.length - 1]);
-    const donePromise = new Promise(resolve => {
-        asm.on('done', asm => {
-            resolve(asm.current);
-        });
-    });
-    await StreamUtil.pipelineAsync(streams);
-    const result = await donePromise;
-    // console.log(JSON.stringify(result, null, 2));
+    let body;
+    const results = await GetDocumentsCommand
+        .parseDocumentsResultResponseAsync(dataStream, store.conventions, _ => body = _);
 }
 ```

src/Http/RavenCommandResponsePipeline.ts

@@ -166,7 +166,21 @@ export class RavenCommandResponsePipeline<TStreamResult> extends EventEmitter {
 
         if (opts.jsonAsync) {
             const parser = new Parser({
-                packKeys: true
+                packKeys: true,
+                packStrings: true,
+                packNumbers: true,
+                streamNumbers: true,
+                streamKeys: false,
+                streamStrings: false
+                // TODO perfect setup perf-wise below, reducing traffic by packing all values 
+                // not yet working due to issue uhop/stream-json#44 - stringer freezes
+                // we still need to stream numbers
+                // packKeys: true,
+                // packStrings: true,
+                // packNumbers: true,
+                // streamNumbers: false,
+                // streamKeys: false,
+                // streamStrings: false
             });
             streams.push(parser);
 

test/Mapping/StreamJsonTests.ts

@@ -73,7 +73,14 @@ describe("stream-json parser and stringer", function () {
         readable.push(null);
 
         const parser = new Parser({
-            packKeys: true
+            packKeys: true,
+            packStrings: true,
+            packValues: true,
+            packNumbers: true,
+            streamNumbers: true,
+            streamValues: false,
+            streamKeys: false,
+            streamStrings: false
         });
 
         const stringerInstance = stringer({ useValues: true });