Node: Expose ability to close GRPC channels

[schmidt-sebastian]

The Firestore Server SDK can initiate multiple clients if more than 100 requests are active. Especially for users that frequently create and destroy clients, this creates memory leaks as we are unable to close GRPC channels on destruction.

Would it be possible to expose a "close" API to the Node Veneer SDKs?

This would unblock googleapis/nodejs-firestore#779 and googleapis/nodejs-firestore#769

@alexander-fenster @thebrianchen

[busunkim96]

👋 Was this issue intended for another repo?

[schmidt-sebastian]

I was hoping this is the right repo. Can you tell me where the code lives that turns the Protos into the auto-generated clients that most GCloud libraries depend on?

[busunkim96]

The generator lives in https://github.com/googleapis/gapic-generator/. @bcoe and @alexander-fenster could you help route this?

[busunkim96]

Ah I see it's already been moved, never mind! 🤦‍♀️

[merlinnot]

@alexander-fenster @schmidt-sebastian Hey guys, maybe we could align in this thread on the design for the API, so I could make a PR? It's very impactful for my company because of the Firestore memory leak and I'd be glad to help fixing it.

[schmidt-sebastian]

I will work on this next week. Please do bug me regularly for updates.