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

[TOOLS-4559] When exporting to local xls files, an error occurs if the data is large. #142

Merged
merged 3 commits into from
Feb 23, 2024

Conversation

hwany7seo
Copy link
Contributor

@hwany7seo hwany7seo commented Feb 22, 2024

http://jira.cubrid.org/browse/TOOLS-4559

Purpose

  • if the data is large (greater the 32767, 'xls file' is not created, and can't know why it wasn't created.

Implementation

  • If the data is large, it is displayed as blank and a list of errors is displayed in the 'no support' tab.

…e data is large.

http://jira.cubrid.org/browse/TOOLS-4559

- If the data is large, it is displayed as blank and a list of errors is displayed in the 'no support' tab.
@hwany7seo hwany7seo self-assigned this Feb 22, 2024
@@ -0,0 +1,72 @@
/*
* Copyright (C) 2009 Search Solution Corporation. All rights reserved by Search Solution.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyright needs to be modified to '2016 CUBRID Corporation'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the advice, fixed it.

@@ -282,14 +284,25 @@ public int writeData(

int index = 0;
for (String val : res) {
sheet.addCell(new jxl.write.Label(index++, total, val));
if (val.length() > 32767) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about using a fixed constant value as a global variable?
ex) private static final int MAX_RECORD_LENGTH

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the advice, fixed it. (MAX_EXCEL_CELL_LENGTH)

@hwany7seo hwany7seo merged commit 57abc99 into CUBRID:develop Feb 23, 2024
2 checks passed
@hwany7seo hwany7seo deleted the xls_large_data branch February 23, 2024 02:10
hwany7seo added a commit to hwany7seo/cubrid-migration that referenced this pull request Feb 23, 2024
…e data is large. (CUBRID#142)

http://jira.cubrid.org/browse/TOOLS-4559

- If the data is large, it is displayed as blank and a list of errors is displayed in the 'no support' tab.
hwany7seo added a commit that referenced this pull request Feb 27, 2024
…e data is large. (#142) (#148)

http://jira.cubrid.org/browse/TOOLS-4559

- If the data is large, it is displayed as blank and a list of errors is displayed in the 'no support' tab.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants