Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not Enough Storage VBA - JSON #61

Open
mnaveedraza opened this issue Jun 14, 2017 · 2 comments
Open

Not Enough Storage VBA - JSON #61

mnaveedraza opened this issue Jun 14, 2017 · 2 comments
Labels

Comments

@mnaveedraza
Copy link

Hi ,

Thank you so much making this JSON Converter. This helps.
However, there were couple of challenges while parsing the JSon data for more than 1 Lakh Records.
Issue 1: "Run Time error for "Not enough storage is available to complete this operation" at this line
Set JS = JsonConverter.ParseJson(http.responsetext)
Issue 2: Unable to extract field names from JSON data. Can you pls help me on how we can do this.

Regards,
Naveed Raza

@Steve184
Copy link

Hi,

JSON Converter is a great library ! It's been working like a charm in one of my programs for every minute of everyday for the past 3 months.

A week or two I also started to encounter the memory issue
"Run Time error for "Not enough storage is available to complete this operation" at this line
Set JS = JsonConverter.ParseJson(http.responsetext)

The program now works 70% of the time. In 30% occasions, it freezes and raises the memory issue.

I have tried everything with no luck (upgrading version, modify options, etc). The only way to restart the program is to kill the process in the task manager... Might be linked to recent windows updates ?

Any idea of what is going on ?

Many thanks for your help !

Steve

@timhall
Copy link
Member

timhall commented Apr 6, 2018

I expected this to be an issue with ConvertToJson since that's creating large strings in memory, but for ParseJson I'm not sure what could be causing it. There is some manual memory management with parsing strings that I had thought about adding a switch for in case it was causing an issue like this, will have to look into further.

@timhall timhall added the bug label Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants