ryanofsky commented on this pull request.
> @@ -138,6 +138,80 @@ class CVectorWriter
+/* Minimal stream for reading from an existing vector by reference
+ const int nType;
+ const int nVersion;
+ const std::vector
I think it would be better to replace VectorReader with SpanReader at this point. It’s a simple change that would make this class more flexible and reusable.