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

Excelize v2.8.1 is causing process restart with performance data #1916

Closed
yflufdvil opened this issue Jun 4, 2024 · 1 comment · Fixed by #1849
Closed

Excelize v2.8.1 is causing process restart with performance data #1916

yflufdvil opened this issue Jun 4, 2024 · 1 comment · Fixed by #1849
Projects

Comments

@yflufdvil
Copy link

Hi,
We are using Excelize v2.8.1 , and we have observed a process restart with this version of Excelize on performance data, same process with same data is working fine with version v2.7.1 ,
It might be a memory leak or some other issue because we have not observed panic error as well, but process is restarting.

Function used -->
excelize.OpenFile
GetRows
Close()

What is the EOL criteria for version v2.7.1
Can you please check?

@xuri
Copy link
Member

xuri commented Jun 6, 2024

Thanks for your issue. This issue was introduced by commit for fix #1692 in v2.8.1, and has been fixed by PR #1849, please upgrade to the master branch code, and this path will be released in the next version. Here is a benchmark for your information. I've closed this. if you have any questions, please let me know.

@xuri xuri closed this as completed Jun 6, 2024
@xuri xuri added this to Performance in v2.9.0 Jun 6, 2024
@xuri xuri linked a pull request Jun 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Performance
v2.9.0
Performance
Development

Successfully merging a pull request may close this issue.

2 participants