Skip to content

Native XML consumes too much memory #9769

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.

Sign up for GitHub

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

Jump to bottom
Open
jonasraoni opened this issue Feb 29, 2024 · 4 comments · May be fixed by pkp/ojs#4799 or #11315
Open

Native XML consumes too much memory #9769

jonasraoni opened this issue Feb 29, 2024 · 4 comments · May be fixed by pkp/ojs#4799 or #11315
Assignees
@Godoy0722
Labels
Enhancement:1:Minor A new feature or improvement that can be implemented in less than 3 days.
Milestone
3.3.0-x

Comments

@jonasraoni
Copy link
Contributor

Describe the bug
According to a user report, the export of an issue took too much memory.

On OJS 3.3 an option to not embed the files as base64 was added, but the following contents are still being embedded:

  • Issue galleys
  • Issue covers
  • Publication covers

This could potentially be the source of the memory issue.

What application are you using?
OJS 3.3
@jonasraoni jonasraoni added the Enhancement:1:Minor A new feature or improvement that can be implemented in less than 3 days. label Feb 29, 2024
@jonasraoni jonasraoni added this to the 3.3.0-x milestone Feb 29, 2024
@cjwetherington
Copy link

cjwetherington commented Mar 14, 2024 •
edited
Loading

We're seeing this on at least 5 of our journals during attempted Native XML exports of individual issues, resulting in 502 server errors. Happy to provide more info if helpful.

@Godoy0722 Godoy0722 self-assigned this Mar 28, 2025
@Godoy0722 Godoy0722 linked a pull request Apr 1, 2025 that will close this issue
Open
@Godoy0722
Copy link
Contributor

Hello @jonasraoni , just a heads up that I just created a PR for this issue. I believe these changes improved the tool's behavior. If you could please take a look, I appreciate that!

@jonasraoni jonasraoni linked a pull request Apr 1, 2025 that will close this issue
create no-embbed option for exporting issues pkp/ojs#4799
Open
Godoy0722 added a commit to Godoy0722/pkp-lib that referenced this issue Apr 24, 2025
@Godoy0722
Add serialization mode for the Native XML import. Needed for issue pk…
ac3a1c7 
…p#9769
@Godoy0722
Copy link
Contributor

Hello @jonasraoni ! If you can, please also take a look here as this PR is also part of this issue. Thank you!

@jonasraoni jonasraoni linked a pull request May 5, 2025 that will close this issue
Add serialization mode for the Native XML import. Needed for issue #9769 #11315
Open
@jonasraoni
Copy link
Contributor Author

As a reminder, an update will need to be done at the PKP PN plugin before the PRs here gets merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Godoy0722 Godoy0722

Labels
Enhancement:1:Minor A new feature or improvement that can be implemented in less than 3 days.
Projects
None yet
Milestone
3.3.0-x
3 participants
@jonasraoni @Godoy0722 @cjwetherington