Skip to content

MODDATAIMP-1095/MODDATAIMP-1255. Properly close resources for File Splitting functionality#371

Merged
SerhiiNosko merged 4 commits intomasterfrom
MODDATAIMP-1095
Dec 16, 2025
Merged

MODDATAIMP-1095/MODDATAIMP-1255. Properly close resources for File Splitting functionality#371
SerhiiNosko merged 4 commits intomasterfrom
MODDATAIMP-1095

Conversation

@SerhiiNosko
Copy link
Contributor

Purpose

Properly close resources for File Splitting functionality
https://folio-org.atlassian.net/browse/MODDATAIMP-1255

Properly close resources for File Splitting functionality
https://folio-org.atlassian.net/browse/MODDATAIMP-1095

Approach

How does this change fulfill the purpose?

TODOS and Open Questions

  • Use GitHub checklists. When solved, check the box and explain the answer.
  • Check logging.

Learning

Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.

@SerhiiNosko
Copy link
Contributor Author

Single file import with Splitting files feature passed
image
Topics contain requestId param
image
image
In logs we also see that new param is present
image

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
76.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@SerhiiNosko SerhiiNosko merged commit b538cf0 into master Dec 16, 2025
5 of 6 checks passed
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.

4 participants