bluesky / databroker

Unified API pulling data from multiple sources
https://blueskyproject.io/databroker
BSD 3-Clause "New" or "Revised" License
33 stars 45 forks source link

Drop unused test infra. #775

Closed danielballan closed 1 year ago

danielballan commented 1 year ago

This removes a module that was an early Databroker 2.x effort to make the JSONL and msgpack tests work. It has never been enabled in the tests because it is very janky. I think we need to look at a migration path for JSONL and msgpack rather than trying to run Tiled on top of them.

A dependency of this code was removed in the just-released v0.1.0a103, so now is the time to excise it.


This PR also changes from msgpack to (zstd-compressed) newline-delimited JSON for GET /documents/{path}. This is symmetric with how documents are uploaded and avoids some confusing issues about chunk-encoded msgpack. We can revisit that can of worms later if we have a reason to.