skyyyy955 opened a new pull request, #10890:
URL: https://github.com/apache/paimon/pull/10890

   ### Purpose
   
   Close the local input stream owned by `FileBasedBloomFilterTest.testProbe`. 
`FileBasedBloomFilter.close()` invalidates its cached page but leaves the 
shared input open, so JUnit cannot delete the temporary file on Windows.
   
   Manage the test input and cache manager with try-with-resources. Keep the 
existing `filter.close()` call and cache assertions inside that scope so 
manager cleanup cannot hide an invalidation failure. This changes one test file 
and adds no production behavior.
   
   ### Tests
   
   Ran the complete `FileBasedBloomFilterTest` on Windows / JDK 8u451 and Linux 
/ JDK 8u441:
   
   | Environment | Before | After |
   | --- | --- | --- |
   | Windows | 7 passed, 1 temporary-directory cleanup error | 8 passed |
   | Linux | 8 passed | 8 passed |
   
   `mvn -o -B -ntp -pl paimon-common -am -DwildcardSuites=none 
-DfailIfNoTests=false -Dtest=FileBasedBloomFilterTest test`
   
   Both final runs passed without `fast-build`. Linux also validates Checkstyle 
outside a `target`-suppressed source path. Full-project tests and official PR 
CI have not run.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to