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

Allow reading json arrays #2028

Closed
scsmithr opened this issue Nov 1, 2023 · 2 comments
Closed

Allow reading json arrays #2028

scsmithr opened this issue Nov 1, 2023 · 2 comments
Labels
duplicate 🥈 This issue or pull request already exists feat 🎇 New feature or request

Comments

@scsmithr
Copy link
Member

scsmithr commented Nov 1, 2023

Description

Currently we only support reading new line delimited json. I'd to also support complete json arrays so that we can read files in the form of:

[
  {
    <object1>
  },
  {
    <object2>
  }
]
@scsmithr scsmithr added the feat 🎇 New feature or request label Nov 1, 2023
@universalmind303
Copy link
Contributor

this should probably happen upstream in either datafusion or arrow-rs.

@tychoish
Copy link
Collaborator

tychoish commented Dec 7, 2023

this should probably happen upstream in either datafusion or arrow-rs.

Likely, though I think I'm inclined to implement this locally, and then try and upstream it, unless there's an obvious way to do this trivally in the upstream.

Also sorry for opening the duplicate issue. I'm inclined to keep the #2218 issue open.

@tychoish tychoish closed this as completed Dec 7, 2023
@universalmind303 universalmind303 added the duplicate 🥈 This issue or pull request already exists label Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate 🥈 This issue or pull request already exists feat 🎇 New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants