Skip to content

Commit e394225

Browse files
dfa1claude
andcommitted
todo: drop VortexException audit item (done)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 0676d90 commit e394225

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

TODO.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ Remaining 0.3 MB gap — biggest to smallest:
9090
- Candidates: `PType` integer kinds, buffer offsets, row indices, byte lengths
9191
- Goal: type-safety at zero cost (value class = no heap alloc, no boxing)
9292

93-
- [ ] **Replace intentional `IndexOutOfBoundsException` throws with `VortexException`**`IndexOutOfBoundsException` is for programming bugs; when bad file data causes an out-of-bounds condition (crafted `bufferIndices`, invalid child index, malformed offset array) the reader should throw `VortexException` so callers can distinguish file corruption from bugs. Audit `buffer(int i)` and `child(int i)` impls in all `Array` subclasses and replace file-triggered throws.
94-
9593
- [ ] **Audit runtime pluggability vs Rust impl** — maintainer (2026-06-04) flagged that Rust supports
9694
runtime registration for: Encodings, DTypes, Compute, Layouts. Java status:
9795
- Encodings: ✅ `ServiceLoader` + `EncodingRegistry.register()`; ✅ `allowUnknown()` passthrough for unregistered encodings (mirrors `VortexSession::allow_unknown()`)

0 commit comments

Comments
 (0)