IGNITE-28870 Restore snapshot temporary files cleanup logic

[oleg-vlsk]

Thank you for submitting the pull request to the Apache Ignite.

In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:

The Contribution Checklist

• There is a single JIRA ticket related to the pull request.
• The web-link to the pull request is attached to the JIRA ticket.
• The JIRA ticket has the Patch Available state.
• The pull request body describes changes that have been made.
  The description explains WHAT and WHY was made instead of HOW.
• The pull request title is treated as the final commit message.
  The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
• A reviewer has been mentioned through the JIRA comments
  (see the Maintainers list)
• The pull request has been checked by the Teamcity Bot and
  the green visa attached to the JIRA ticket (see tab PR Check at TC.Bot - Instance 1 or TC.Bot - Instance 2)

Notes

• How to Contribute
• Coding abbreviation rules
• Coding Guidelines
• Apache Ignite Teamcity Bot

If you need any help, please email dev@ignite.apache.org or ask anу advice on http://asf.slack.com #ignite channel.

[oleg-vlsk]

/runall

[anton-vinogradov]

@oleg-vlsk that looks like an Ignite PR Checker command — but the checker doesn't know your accounts yet, so nothing was triggered. Two steps to make it work:

1. Log in at https://ignite-pr-checker.is-a.dev with your TeamCity (ci2) access token.
2. In settings (⚙) switch on Comment my runs' verdicts on the GitHub PR (takes a GitHub personal access token, public_repo scope).

After that, commenting /run-all (or /run-all top) here queues the whole RunAll chain under your own TeamCity account, this thread gets a live ETA, and the verdict lands in the PR when the run settles — JIRA visas and automatic re-runs of blocker suites are separate switches next to it.

[oleg-vlsk]

/runall

---

🚀 RunAll queued — build 9201603. The verdict lands here when the run finishes.
🏁 Run finished — the verdict comment has the full story.

[oleg-vlsk]

Ignite PR Checker verdict · RunAll build 9201603 · 147 suites ran, 0 reused

⚠️ 1 broken suite(s) (no reliable run):

• Platform .NET (Windows) 3: execution timeout · non-zero exit code

❌ 4 blocker(s) in 4 suite(s):

• Platform .NET (Windows): Apache.Ignite.Core.Tests.exe: Apache.Ignite.Core.Tests.Compute.Forked.ForkedBinarizableClosureTaskTest.TestAppylMultipleReducerJobException
• Cache 10: org.apache.ignite.testsuites.IgniteCacheTestSuite10: org.apache.ignite.internal.processors.cache.IgniteTxConfigCacheSelfTest.testUserTxTimeout
• Snapshots 6: org.apache.ignite.testsuites.IgniteSnapshotTestSuite6: org.apache.ignite.internal.processors.cache.persistence.snapshot.incremental.IncrementalSnapshotRestoreTest.testRecoveryWithNotBaselineNode
• Snapshots 8: org.apache.ignite.testsuites.IgniteSnapshotTestSuite8: org.apache.ignite.internal.processors.cache.persistence.snapshot.IgniteClusterSnapshotCheckTest.testSnapshotCheckMetricsLesserTopology[encryption=true, onlyPrimary=false, snpThrdPoolSz=4]

[oleg-vlsk]

Ignite PR Checker verdict · RunAll build 9224988 · 147 suites ran, 0 reused

⚠️ 2 broken suite(s) (no reliable run):

• Platform C++ CMake (Win x64 | Release): IgniteCoreTest terminated abnormally - the test binary died, its remaining tests never ran · Number of tests 458 is 23% less than 595 in build #18247
• Cache (Failover) 5: execution timeout · non-zero exit code · Number of tests 19 is 72% less than 67 in build IGNITE-1835 .Net: Use thread-local pool to store schemas in PortableWriter #196

🔍 3 suite(s) ran fewer tests than on master (tests that never ran can't fail):

• Cache (Failover) 5: 19 tests vs 67 on master (−72%)
• Platform C++ CMake (Linux): 595 tests vs 1046 on master (−43%)
• Platform C++ CMake (Linux Clang): 595 tests vs 1046 on master (−43%)

✅ No test blockers otherwise; 29 pre-existing/flaky filtered out.

[oleg-vlsk]

Ignite PR Checker verdict · RunAll build 9232787 · 147 suites ran, 0 reused

⚠️ 2 broken suite(s) (no reliable run):

• Open Census: Number of tests 3 is 96% less than 82 in build #16625
• Cache 6: execution timeout · non-zero exit code · Number of tests 112 is 34% less than 169 in build #45382

🔍 1 suite(s) ran fewer tests than on master (tests that never ran can't fail):

• Cache 6: 112 tests vs 170 on master (−34%)

✅ No test blockers otherwise; 22 pre-existing/flaky filtered out.