[Core] Liquigraph crashes when trying to change schema (add index / create constraint) about liquigraph HOT 9 CLOSED

Thanks for the report, we'll look into it.

from liquigraph.

Bingo: https://github.com/fbiville/liquigraph/blob/master/liquigraph-core/src/main/java/org/liquigraph/core/writer/ChangelogGraphWriter.java#L51.

I will push a fix this week.

from liquigraph.

Fabulous! thanks a lot! I'll be going on holiday soon so no rush, I'll test
it again when I'm back in June :)

On Tue, 12 May 2015 08:38 @fbiville [email protected] wrote:

Bingo:
https://github.com/fbiville/liquigraph/blob/master/liquigraph-core/src/main/java/org/liquigraph/core/writer/ChangelogGraphWriter.java#L51
.

I will push a fix this week.

—
Reply to this email directly or view it on GitHub
https://github.com/fbiville/liquigraph/issues/40#issuecomment-101167502.

from liquigraph.

Hi @fbiville. Just saw this great tool and looking at incorporating it into our project. This fix and https://github.com/fbiville/liquigraph/issues/23 would be great to include - any idea when RC3 might be ready to cut?

from liquigraph.

@mbyrne00 RC3 will hopefully be out within the next 2 weeks. Would that timing suit you?

from liquigraph.

That would be great! In the mean time when #23 has been merged into master we could build snapshot locally and give it a rest to provide feedback.

from liquigraph.

@mbyrne00 #23 is a bit unsatisfactory at that point, because there is still a time window during which a concurrent execution may happen, as described here: https://groups.google.com/forum/?hl=en-GB#!topic/neo4j/28t4xOlhGLg.

We can still merge it now and improve it later.

from liquigraph.

Yea it would be great to have what's there to date. I'd like to give it a test and look under the covers. Thanks!

from liquigraph.

Fixed in master.

from liquigraph.