Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert-store #22

Open
wants to merge 37 commits into
base: revert-store
Choose a base branch
from
Open

Revert-store #22

wants to merge 37 commits into from

Conversation

snehabanerjee28
Copy link

Creating pull request for neo4j upgrade

jexp and others added 30 commits September 28, 2014 22:57
The StoreCopy tool's output formatting doesn't behave reliably when
there's a lot of compaction going on.  A lot of the '.'s are skipped and
the line-ending summaries are skipped when a node or relationship
with the current node ID or rel ID doesn't exist (because that
information is outputted after getting the node / relationship).  This
resolves that so that every line is approximately the same length and
there's a line per every 500000 nodes/relationships.

This change also ups the number of nodes per '.' and per line.
Clean up StoreCopy output formatting
similar to copyRelationships method, try-catch block is added
to skip faulty node records, and continue with copying.
added try-catch block in copyNodes methods
* 22:
  StoreComparer: add missing transaction wrapping.
  better error handling
StoreComparer: add missing transaction wrapping
#15)

This is needed because during shutdown batch inserter tries to flush it's
caches and some entities in those caches might be corrupt for the source
database.

Problem initially seen for unused dynamic label records.
- allow for labels to be specified that cause their nodes to be deleted
- allow for relationships to be skipped if their nodes have not been copied
- allow for specification if node-ids should be compacted
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants