Memory Leak on readStream

Lib version 3.3.1

Here is example

const {
  EventStoreDBClient,
  jsonEvent,
  START,
  FORWARDS
} = require("@eventstore/db-client")

var heapdump = require('heapdump')

const client = new EventStoreDBClient(
  {
    endpoint: "localhost:2113",
  },
  {
    insecure: true,
  }
);

(async () => {
  const streamName = "es_supported_clients"
  const event = jsonEvent({
    type: "grpc-client",
    data: {
      languages: ["typescript", "javascript"],
      runtime: "NodeJS",
    },
  })
  await client.appendToStream(streamName, [event])

  for (let index = 0; index < 10000; index++) {
    const events = client.readStream(streamName, {
      fromRevision: START,
      direction: FORWARDS,
      maxCount: 10,
    })

    // if remove the loop. memory leak even bigger
    for await (const event of events) {
    }
    global.gc()
    console.log(
      // convert to mb
      Math.round((process.memoryUsage().heapUsed / 1024 / 1024) * 100) / 100
    )
  }
  heapdump.writeSnapshot(function (err, filename) {
    console.log('dump written to', filename)
  })
})()

[George-Payne]

@matchish v4.0.0 has been released, including this fix.

https://www.npmjs.com/package/@eventstore/db-client/v/4.0.0 https://github.com/EventStore/EventStore-Client-NodeJS/releases/tag/v4.0.0

[George-Payne]

Thanks @matchish, good find.

The fix / bit of explanation is here: #310