Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862905437 Fixed. The issue was that file keys are optional in the java file attributes. Adding a fallback to the path makes WindowsFS also work on Windows so we can apply WindowsFS also

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler closed issue #11920: Test testDeleteUnusedFiles() failed in TestIndexWriter URL: https://github.com/apache/lucene/issues/11920 -- 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

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler closed issue #11920: Test testDeleteUnusedFiles() failed in TestIndexWriter URL: https://github.com/apache/lucene/issues/11920 -- 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

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862308528 Here is a PR that uses toRealPath as key if the fileKey is not provided (null). I was able to remove all "does not work on Windows assumes" in TestIndexWriter. -- This is an a

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862240965 If we fallback to the path, we do not need to add special cases for file renames. The current code checks if the filekey has changed after rename and fixes its internal structure

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862192925 @rmuir do you have any other ideas - you were writing WindowsFS. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862190434 I was able to fix the whole test by doing the following: - In WindowsFS fall back to path as key, if the file key isn't available. I have no yet figured out if this brings othe

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-08 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2862170475 I am debugging. The reason for the whole failure is some spaghetti conditions that are hard to understand. Actually WindowsFS does not work with windows, as the Windows imp

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-07 Thread via GitHub
dweiss commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2861906753 > By the way, my new PC's performance is so awsome, ./gradlew check only takes 1m 42s It also shows that something has gone terribly wrong with gradle... Eh. -- This is an

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-07 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2860592088 On GitHub it does not fail as "windows-latedt" Tag is Windows Server 2022, which is basically Windows 10 kernel. To reproduce on GitHub we need to change job config to use

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-07 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2860543452 I think the problem is that Windows 11 changed semantics and allows to delete open files under some circumstances. The windows emulation is not enabled on Windows 11 so thi

Re: [I] Test testDeleteUnusedFiles() failed in TestIndexWriter [lucene]

2025-05-07 Thread via GitHub
uschindler commented on issue #11920: URL: https://github.com/apache/lucene/issues/11920#issuecomment-2860504381 Hi, Since Jenkins was updated to Windows 11 every build fails with same message. My local box is also Windows 11 and fails in same way. We should fix this. The pro