Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FileReftableStack.addReftable does not match the specifications #136

Open
zotttelbart opened this issue Jan 29, 2025 · 0 comments
Open

FileReftableStack.addReftable does not match the specifications #136

zotttelbart opened this issue Jan 29, 2025 · 0 comments

Comments

@zotttelbart
Copy link

Version

7.1.0

Operating System

Windows

Bug description

The implementation of FileReftableStack.addReftable doesn't match the specifications in FileReftableStack.addReftable

The specification says, that a writer have to read the tables.list files to detemine the current tables after the lock is acquired.

Actual behavior

addReftable fails, if the stack is not upToDate.

Expected behavior

After acquire the lock for tables.list the stack should be reload if necessary.

Relevant log output

Other information

With change https://eclipse.gerrithub.io/c/eclipse-jgit/jgit/+/1206683 @msohn try to reload the stack, if necessary. But I don't know, if it is enough in racy conditions. To meet the specification addReftable should call reload or reloadIfNecassary.

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

No branches or pull requests

1 participant