PR #1138 fixes a real TypeScript bug around integer PhysicalLevelTechnique.NONE, but while working on it I found that the repository is not aligned on whether plain integer streams are actually part of the supported design.
Current state:
docs/specification.md lists integer physical techniques as SIMD-FastPFOR and Varint
docs/encodings.md still defines Plain integer and long payloads as little-endian byte streams
- Rust supports and emits plain integer streams (
PhysicalEncoder::None)
- Java integer decoding only supports
FAST_PFOR and VARINT
- the generic C++ integer decoder rejects
NONE, although geometry code has a partial tolerance path
So the question is: what is the intended project-wide direction here?
- Keep plain integer streams as supported
- Remove them from the supported design
- Other / mixed approach
PR #1138 fixes a real TypeScript bug around integer
PhysicalLevelTechnique.NONE, but while working on it I found that the repository is not aligned on whether plain integer streams are actually part of the supported design.Current state:
docs/specification.mdlists integer physical techniques asSIMD-FastPFORandVarintdocs/encodings.mdstill definesPlaininteger and long payloads as little-endian byte streamsPhysicalEncoder::None)FAST_PFORandVARINTNONE, although geometry code has a partial tolerance pathSo the question is: what is the intended project-wide direction here?