Memory leak is still happen in both of newest version and prior version. about node-red-contrib-mssql-plus

[BADEnder]

@adikos The version I modified is version 0.3.3 as the picture above, not version 0.7.3

[adikos]

Hi @BADEnder,
I'm facing the same memory leakage with 0.7.3 version. Can you upload your corrected version to give it a try?
Thanks.

[uoral]

Hello Mr. @Steve-Mcl ,

I'm facing that memory leakage problem too. Please update that version. Thank you so much.

[uoral]

Hi @BADEnder ,

How can we modify that code?. Can you please explain that. With a program? I deleted the parts in the picture from the mssql.js file, but it didn't work. Please help me. Thank you.

[uoral]

Hello Mr @bestlong ,

Because of this problem(constant loss of ram) my program is not working properly and after a while the program gets blocked. Please a solution here. Thank you so much.

Kind regards.

[bestlong]

Hi @BADEnder @PyregTechnologie

Now node can disable parse mustache.

[uoral]

Hi Mr. @bestlong,

First of all, thank you very much for your quick response. Do we need to reinstall Mssql-plus Node? How can I implement this change in the code?

Kind Regards

[BADEnder]

@PyregTechnologie

If you use the version 0.3.3, you can just replace the mssql.js in "src" directory with the link below:

https://github.com/BADEnder/node-red-contrib-mssql-plus-m/blob/master/mssql.js

Not only delete the mustache and others, but also add some code referred to the module named "node-mssql"

[bestlong]

Hi @PyregTechnologie

First of all, thank you very much for your quick response. Do we need to reinstall Mssql-plus Node? How can I implement this change in the code?

Kind Regards

go to Manage palette update node and restart nodered.

then edit node property uncheck parse mustache

[uoral]

Hi @BADEnder @bestlong ,

Thank you very much for your answers and solutions.

Kind Regards

[bestlong]

If you use the version 0.3.3, you can just replace the mssql.js in "src" directory with the link below:

https://github.com/BADEnder/node-red-contrib-mssql-plus-m/blob/master/mssql.js

Not only delete the mustache and others, but also add some code referred to the module named "node-mssql"

Hi @BADEnder

We can't see node-red-contrib-mssql-plus-m. Private repository ?

[BADEnder]

Sorry, that's my fault, and it's public now.
If you have any concern about the code, please let me know, thanks.

[BADEnder]

Hi @bestlong, what do you think after reviewed my modified code?

By the way, I test your newest version 0.10.x this Monday, and still got the memory leak.

In my opinion, maybe that's about the code creating a new promise object in node-red instead of using the promise object that's already built in the module "node-mssql".
Therefore, I guess that the memory will keep going up if you don't delete those object to release the memory.