BlockingQueue implementation and other interfaces

[pschichtel]

BlockingQueue implementation and other interfaces

[JakeWharton]

The main reason is that the queue is not safe for concurrent interaction. A quick glance at the BlockingQueue docs and your gist makes it seem like you are not fulfilling the contract of the interface:

BlockingQueue implementations are thread-safe.

[pschichtel]

All operations to the backing QueueFile are synchronized, so there is no concurrent access to the QueueFile (unless of course someone shares the QueueFile instances to other places). This is similar to the ArrayBlockingQueue which synchronizes every operation to the backing array.
I've missed the locking in the add() method, but that's fixed locally.

[JakeWharton]

Ah, haha. The add method was the only one I looked at!

I don't think I'm opposed to adding this if you're willing to send a PR and write tests.

[pschichtel]

Is this project still alive?

[pforhan]

Used a lot of places but not heavily developed as it's pretty mature. I think there's a branch with a 2.0 proposal. -- https://github.com/square/tape/tree/2.0.0-RC1