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

fixes #1257: apoc.load.json file roundtrip from apoc.export.json.all #1409

Merged
merged 1 commit into from Apr 8, 2020

Conversation

conker84
Copy link
Contributor

Fixes #1257

Imported json files produced with apoc.export.json.all

Proposed Changes (Mandatory)

A brief list of proposed changes in order to fix the issue:

  • created apoc.import.json.all
  • added tests
  • updated documentation

@JMHReif
Copy link
Contributor

JMHReif commented Mar 12, 2020

Is the difference between this and apoc.load.json just that it takes maps with both nodes/rels in same json file? Should we move the procedure under the apoc.import.* or apoc.load.json* path?

@jexp
Copy link
Member

jexp commented Mar 13, 2020

It's meant to support the roundtrip from apoc.export.json from another db.

The json package should be fine.

I don't think we need the .all suffix.

@conker84
Copy link
Contributor Author

It's meant to support the roundtrip from apoc.export.json from another db.

The json package should be fine.

I don't think we need the .all suffix.

@jexp I removed the .all suffix

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.

apoc.load.json file roundtrip from apoc.export.json.all
4 participants