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

Bun taking too long to load the workbook and write to it #14034

Open
yashone7 opened this issue Sep 19, 2024 · 1 comment
Open

Bun taking too long to load the workbook and write to it #14034

yashone7 opened this issue Sep 19, 2024 · 1 comment
Labels
bug Something isn't working needs triage

Comments

@yashone7
Copy link

What version of Bun is running?

1.1.28+cf4e9cb69

What platform is your computer?

Microsoft Windows NT 10.0.22631.0 x64

What steps can reproduce the bug?

https://github.com/yashone7/bun-artifacts-sync

Here is the repo. Copy the excel file in the repo and initialize a git repo and commit something in it.
Run bun index.js and make changes to this excel file and save it. The scripts runs and gets the commit_id of the git repo you have initialized where the excel file is and copies it into a dist folder and reads the commit_id, loads the workbook and writes the commit_id to the file.

As you can see here bun is much slower than nodejs in this case.

Screenshot 2024-09-19 125546 ![Screenshot 2024-09-19 125701](https://github.com/user-attachments/assets/295f526a-e04d-485d-9ca8-0bcec484381d)

What is the expected behavior?

Reading and Writing excel files by exceljs should be within acceptable range. Right now it takes 30 seconds to read and 30 seconds to write. In comparison nodejs takes about 30ms to read and write each.

What do you see instead?

Screenshot 2024-09-19 125701
Screenshot 2024-09-19 125546

Additional information

No response

@yashone7 yashone7 added bug Something isn't working needs triage labels Sep 19, 2024
@carlos-duran
Copy link

I'm using exceljs in production, I never thought that Bun was slower than node in this library.
My microservice is using Bun.file and others Bun APIs 🙁 It will be difficult, but I think I have to do a POC with Node.

Thanks for showing these benchmarks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage
Projects
None yet
Development

No branches or pull requests

2 participants