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

Temporary files not cleaned up after Maven plugin execution #22108

Closed
ancoron opened this issue Jun 25, 2020 · 3 comments
Closed

Temporary files not cleaned up after Maven plugin execution #22108

ancoron opened this issue Jun 25, 2020 · 3 comments
Labels
status: superseded An issue that has been superseded by another

Comments

@ancoron
Copy link

ancoron commented Jun 25, 2020

Whenever executing a Maven project build (with or without tests - doesn't matter), exactly two temporary files are left in place without any content, e.g.:

-rw-rw-r-- 1 ancoron ancoron 0 Jun 25 14:23 /tmp/springboot-11563970946139969408-entrycontent
-rw-rw-r-- 1 ancoron ancoron 0 Jun 25 14:23 /tmp/springboot-5070898860288932403-entrycontent

The only reference I found to this pattern was from the SizeCalculatingEntryWriter.

As everyone can see, using the temporary file is optional, although created every time and there is no safe-guard that would delete those files on JVM exit.

This means that over time, the temporary directory gets flooded with empty files.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 25, 2020
@wilkinsona wilkinsona changed the title [maven-plugin] Temporary files not cleaned up after plugin execution Temporary files not cleaned up after Maven plugin execution Jun 25, 2020
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 25, 2020
@wilkinsona wilkinsona added this to the 2.3.x milestone Jun 25, 2020
dreis2211 added a commit to dreis2211/spring-boot that referenced this issue Jun 25, 2020
@wilkinsona
Copy link
Member

Closing in favour of #22112.

@wilkinsona wilkinsona removed this from the 2.3.x milestone Jun 26, 2020
@wilkinsona wilkinsona added the status: superseded An issue that has been superseded by another label Jun 26, 2020
@wilkinsona wilkinsona removed the status: superseded An issue that has been superseded by another label Jul 9, 2020
@wilkinsona wilkinsona added this to the 2.3.x milestone Jul 9, 2020
@wilkinsona wilkinsona reopened this Jul 9, 2020
@wilkinsona
Copy link
Member

Looking at this more closely, I'd like to understand the need for the temporary file. Presumably there were entries of an unknown size that are too large to be dealt with in memory. If that's the case, I'd expect them to also be too large when writing unpack comments as it currently does everything in memory as well. It may be that, as the unpack comments are opt-in, we just haven't seen the problem yet.

@wilkinsona wilkinsona added the for: team-attention An issue we'd like other members of the team to review label Jul 9, 2020
@philwebb
Copy link
Member

Closing in favor of PR #22112. I've also opened #22305 to see if we can find a better long-term solution.

@philwebb philwebb removed this from the 2.3.x milestone Jul 10, 2020
@philwebb philwebb added status: superseded An issue that has been superseded by another and removed for: team-attention An issue we'd like other members of the team to review type: bug A general bug labels Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

No branches or pull requests

4 participants