Use GraphAware TX API + Shell command for index import

neo4j-contrib / neo4j-elasticsearch

Neo4j ElasticSearch Integration

Apache License 2.0

210 stars 79 forks source link

Use GraphAware TX API + Shell command for index import #8

Open jazzido opened 9 years ago

jazzido commented 9 years ago

Hey @jexp,

There's two major changes in this PR:

I'm now using @graphaware framework for implementing the transaction change listener.
Initial version of a shell command for importing nodes into an ElasticSearch index (#7)

Cheers,

jexp commented 9 years ago

I'd rather not pull in an external dependency at this point, as I think this should stay a minimal demonstration on how to achieve this kind of integration.

But if you think this makes sense, I'm more than happy to link your fork from the developer pages too as one example how to take this further.

What do you think?

jazzido commented 9 years ago

Sure. I'm willing to maintain my fork, as I'm using this in production and not merely as a demonstration. I also tried to avoid introducing a new dependency, but @graphaware's improved transaction classes allowed me to track node changes in the transaction handler without writing a lot of new (and potentially buggy) logic.

I'd appreciate if you could link to my fork from the README.

Thanks!