Skip to content

Bugfix for 66104 - Check for the existence of the log directory before executing the cleanup task at startup#521

Closed
aooohan wants to merge 1 commit into
apache:mainfrom
aooohan:fixbug_66104
Closed

Bugfix for 66104 - Check for the existence of the log directory before executing the cleanup task at startup#521
aooohan wants to merge 1 commit into
apache:mainfrom
aooohan:fixbug_66104

Conversation

@aooohan

@aooohan aooohan commented Jun 5, 2022

Copy link
Copy Markdown
Member

markt-asf added a commit that referenced this pull request Jun 6, 2022
Avoid error message by not trying to clean up old files from the logging
directory before the directory has been created.
Based on #521 by HanLi.

Refactor obtaining Path for a given directory
markt-asf added a commit that referenced this pull request Jun 6, 2022
Avoid error message by not trying to clean up old files from the logging
directory before the directory has been created.
Based on #521 by HanLi.

Refactor obtaining Path for a given directory
markt-asf added a commit that referenced this pull request Jun 6, 2022
Avoid error message by not trying to clean up old files from the logging
directory before the directory has been created.
Based on #521 by HanLi.

Refactor obtaining Path for a given directory
markt-asf added a commit that referenced this pull request Jun 6, 2022
Avoid error message by not trying to clean up old files from the logging
directory before the directory has been created.
Based on #521 by HanLi.

Refactor obtaining Path for a given directory
@markt-asf

Copy link
Copy Markdown
Contributor

Thanks for the bug report and the PR.

I applied this manually so I could:

  • refactor this code and the other location that obtains a Path from the directory
  • used a Java 7+ compatible approach that could be back-ported and then changed this to the Java 11+ approach
  • add a changelog entry

@markt-asf markt-asf closed this Jun 6, 2022
@aooohan aooohan deleted the fixbug_66104 branch June 6, 2022 12:09
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.

2 participants