Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Read JSON Lines format #419

Closed
tznind opened this issue Jun 22, 2021 · 2 comments · May be fixed by #422
Closed

Read JSON Lines format #419

tznind opened this issue Jun 22, 2021 · 2 comments · May be fixed by #422

Comments

@tznind
Copy link
Contributor

tznind commented Jun 22, 2021

Is your feature request related to a problem? Please describe.
A DLE attacher that can read .json and populate RAW tables is sought.

If we can convert json to System.DataSet using a library then this should be very easy

@jas88
Copy link
Member

jas88 commented Jun 22, 2021

@tznind
Copy link
Contributor Author

tznind commented Jun 23, 2021

Sounds like a job for https://www.nuget.org/packages/System.Text.Json/6.0.0-preview.5.21301.5?

I think we already have a dependency on using Newtonsoft.Json; and it looks like it should be relatively easy to get the data out.

Then we either need to put it into a System.DataSet or a collection of System.DataTable and use the bulk insert feature of FAnsiSql or go the slower route of calling DiscoveredTable.Insert(Dictionary<DiscoveredColumn,object> values).

Hopefully we don't need to deal with auto increment because that is a no go for RDMP DLE.

Nice intro reading:
https://docs.microsoft.com/en-us/sql/relational-databases/json/json-data-sql-server?view=sql-server-ver15

@HicServices HicServices locked and limited conversation to collaborators Sep 23, 2021
@tznind tznind closed this as completed Sep 23, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants