Skip to content

Fix recovery issue on windows.#60

Open
RisbergAdam wants to merge 1 commit into
yahoo:masterfrom
RisbergAdam:master
Open

Fix recovery issue on windows.#60
RisbergAdam wants to merge 1 commit into
yahoo:masterfrom
RisbergAdam:master

Conversation

@RisbergAdam

Copy link
Copy Markdown

Fixes #23. On Windows, if any file specified in the call to Files.move is open by any process (including our own), the call will throw a FileSystemException.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@RisbergAdam

Copy link
Copy Markdown
Author

@bellofreedom maybe you can take a look at this PR?

@RisbergAdam

Copy link
Copy Markdown
Author

Any updates on this PR?

darkfrog26 added a commit to outr/HaloDB that referenced this pull request Jun 4, 2026
…ndows

Ports yahoo#60 (fixes #23): Files.move fails with FileSystemException on
Windows if the source or target is held open. Closing the index and data files
before renaming over them is required there and harmless on POSIX.
@darkfrog26

darkfrog26 commented Jun 4, 2026

Copy link
Copy Markdown

Thanks for this fix! HaloDB upstream looks unmaintained, so I've started a maintained fork — outr/HaloDB, published to Maven Central as com.outr:halodb-revive. I've ported this change into it (with credit). Sharing in case it's useful to anyone landing here.

Update: the fork has since grown — it now supports arbitrary-length keys (the 127-byte cap is gone) and prefix/range scans, alongside this Windows recovery fix.

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.

Got FileSystemException when repairing database

2 participants