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

fix: big xls file error #4859

Merged
merged 3 commits into from
Feb 12, 2025
Merged

Conversation

SkyfireWXY
Copy link
Contributor

@SkyfireWXY SkyfireWXY commented Feb 10, 2025

What problem does this PR solve?

if *.xls file is too large, .eg >50M, I get error.

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

@KevinHuSh
Copy link
Collaborator

Appreciation!
Please keep 4 spaces as indent.

@SkyfireWXY
Copy link
Contributor Author

sure, 4 spaces as indent done:)

@SkyfireWXY
Copy link
Contributor Author

Appreciation! Please keep 4 spaces as indent.

i clicked "Merge branch 'main' into bugfix/issue_4856". is it good?

@yingfeng yingfeng added the ci Continue Integration label Feb 11, 2025
@KevinHuSh KevinHuSh merged commit 8fcca1b into infiniflow:main Feb 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continue Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants