Skip to content

Commit 37cb508

Browse files
dfa1claude
andcommitted
docs: update TODO — remove completed DeltaEncoding item, detail benchmark tasks
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8108034 commit 37cb508

1 file changed

Lines changed: 12 additions & 7 deletions

File tree

TODO.md

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,18 @@
1111
and benchmark commit SHA so numbers don't rot silently.
1212

1313
- [ ] **#13 Cascading compressor — remaining follow-ups**
14-
- [ ] **DeltaEncoding wire-format fix**: refactor to emit `(bases child, deltas child)` matching
15-
Rust's 2-child wire format (currently 1-buffer flat — incompatible with Rust reader, see commit 09685a2).
16-
Unblocks cross-compat for delta cascades.
17-
- [ ] **Cross-compat test**: `JavaWritesRustReadsIntegrationTest` with `allowedCascading=3`
18-
confirms Rust JNI reader decodes the same OHLC data (gates DeltaEncoding wire-format fix).
19-
- [ ] **Benchmark**: extend `RustVsJavaWriteBenchmark` with cascading-off vs cascading-on variants;
20-
capture compressed-bytes ratio (Java cascade / JNI), write throughput, decode throughput.
14+
- [ ] **Cross-compat test**: Add `JavaWritesRustReadsIntegrationTest` in the `integration` module.
15+
Write OHLC data with `WriteOptions.cascading(3)` via Java writer; read it back with the Rust JNI
16+
reader and assert values round-trip correctly. Same OHLC schema / dataset as `RustVsJavaWriteBenchmark`.
17+
- [ ] **Benchmark — cascading write variants**: In `RustVsJavaWriteBenchmark`, add:
18+
- `javaWriteCascading()` — same body as `javaWrite()` but use `WriteOptions.cascading(3)` instead of `WriteOptions.defaults()`
19+
- After both `javaWrite` and `javaWriteCascading` run, log the compressed-bytes ratio
20+
(`javaWriteCascading file size / javaWrite file size`) via a JMH `AuxCounters` or a
21+
`@TearDown` print so it appears in the benchmark output.
22+
- [ ] **Benchmark — cascading read variants**: In `RustVsJavaReadBenchmark`, add a
23+
`javaReadCascading()` method that reads a file written with `WriteOptions.cascading(3)`.
24+
Warm up the file in `@Setup`; measure decode throughput (rows/s). Compare against `javaRead()`
25+
(no cascading) and `jniRead()` to show the cascading cost/benefit at read time.
2126

2227
## Tooling
2328

0 commit comments

Comments
 (0)