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

Considering streaming output #15

Closed
ThomasG77 opened this issue Nov 17, 2017 · 3 comments
Closed

Considering streaming output #15

ThomasG77 opened this issue Nov 17, 2017 · 3 comments

Comments

@ThomasG77
Copy link
Contributor

ThomasG77 commented Nov 17, 2017

Like the previous issue I opened #14, I would like to be able to use streaming output like http://ndjson.org
Why? it's about partial consumption of data and lowering footprint on the back-end as not streaming = output size = 600Mo and RAM consumption on the server = 600Mo

@cportele
Copy link
Member

I would expect that most implementations will support "streaming", i.e. return features in a feature collection in chunks (e.g. feature by feature). Currently that is not discussed, but I agree that we should say something about it.

And we should mention that there is a standard for streaming GeoJSON: GeoJSON Text Sequences. This is similar to ndjson, but uses a different separator convention. I have not seen it used a lot, so we should first have more implementation experience before considering a conformance class.

@jdesboeufs
Copy link

jdesboeufs commented Nov 17, 2017

It's mostly an implementation concern (from database to final client).
Many formats are streaming friendly: GeoJSON, Geobuf, CSV, GML…

In JS world we have:
https://github.com/dominictarr/JSONStream
https://github.com/node-geojson/geojson-stream (built upon JSONStream)

ndjson is interesting when you have to deal with line-by-line parsing (bash/grep, Python stdlib), but it is not very used.

@cportele
Copy link
Member

Discussed during the meeting on 2017-11-28: The media type (and compression options etc) will determine the way the response is returned. The HTTP mechanisms can be used. Add a statement in the document, but no normative statement seems required.

@cportele cportele self-assigned this Nov 28, 2017
@cportele cportele added this to the Part 1, First Draft milestone Nov 28, 2017
cportele added a commit that referenced this issue Dec 14, 2017
cportele pushed a commit that referenced this issue Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants