OCFL / ocfl-java

A Java OCFL implementation
MIT License
16 stars 12 forks source link

Revert "make all file operations synchronous" #93

Closed pwinckles closed 1 year ago

pwinckles commented 1 year ago

This reverts commit 39c09e22cd946b618764ad623d3c3d63598565d6.

I'm reverting the change that made some of the write operations synchronous for now, because it turns out copies have the same problem and the standard library does not expose an option for making them synchronous. Additionally, it was reported that making the writes synchronous had a significant performance impact. So, if we decide to add synchronous writes, they must be opt-in, and they must be applied to all operations, including those that the standard library does not offer an easy way to make synchronous.