disqus / pgshovel

A change data capture system for PostgreSQL
Apache License 2.0
11 stars 3 forks source link

Relay should ensure it is running version that is compatible with the origin database. #20

Open tkaemming opened 9 years ago

tkaemming commented 9 years ago

This is partially implemented already.

Fluxx commented 9 years ago

@tkaemming could you elaborate on the specifics of the race condition?

tkaemming commented 9 years ago

I don't really recall — it was causing a deadlock while running tests (but worked fine in practice.) I'd imagine that uncommenting it would cause the same behavior if it's still problematic.

Fluxx commented 9 years ago

Alright, will uncomment it and see what happens!

Fluxx commented 9 years ago

I just commented the code out and tests passed without issue. So I'm gonna put up a PR with it removed.