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

https://www.pivotaltracker.com/story/show/79728226 #494

Merged
merged 1 commit into from Oct 1, 2014

Conversation

osmandin
Copy link
Contributor

@osmandin osmandin commented Oct 1, 2014

The specific functionality that's causing the symptom noted in PT # 79728226 was introduced via commit 068c346

Rationale for the PR:

ReadOnlyExternalPropertiesFedoraFileSystemConnectorIT fails when trying to delete a file generated when running
BasicReadWriteFedoraFileSystemConnectorIT. The former makes no use of this file, and hence shouldn't try to delete it (indirectly via AbstractFedoraFileSystemConnectorIT).

This fixes the specific build error that result from attempting to delete a file that's still held on by another Java process. The file lock was observed on Windows Java 7u67 (and 7u25). I used the Windows handle command to see that the file was indeed being held open by a java process. I was unable to get further details about which thread was holding the file open. As this seems to be a common problem on Windows, it's not clear to me whether this is worth investigating.

The specific functionality that's causing the symptom noted in PT # 79728226 was introduced via commit fcrepo@068c346

Rationale for the PR:

ReadOnlyExternalPropertiesFedoraFileSystemConnectorIT fails when trying to delete a file generated when running
BasicReadWriteFedoraFileSystemConnectorIT. The former makes no use of this file, and hence shouldn't try to delete it (indirectly via AbstractFedoraFileSystemConnectorIT).

This fixes the specific build error that result from attempting to delete a file that's still held on by another Java process. The file lock was observed on Windows Java 7u67 (and 7u25). I used the Windows handle command to see that the file was indeed being held open by a java process. I was unable to get further details about which thread was holding the file open. As this seems to be a common problem on Windows, it's not clear to me whether this is worth investigating.
@awoods awoods merged commit 4e33fd2 into fcrepo:master Oct 1, 2014
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.

None yet

2 participants