Skip to content

Commit 0f136da

Browse files
dfa1claude
andcommitted
fix(core): decode string/bytes scalar values in ArrayStats
Enables utf8 column min/max to appear in zone-map stats and vortex stats output. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 1fc7e53 commit 0f136da

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

TODO.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030

3131
## Tooling
3232

33-
- [ ] String min/max in stats — requires utf8 scalar encoding in the proto
3433

3534
## Large-file support
3635

core/src/main/java/io/github/dfa1/vortex/core/ArrayStats.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,8 @@ private static Object decodeScalar(ByteBuffer bytes) {
4343
case F32_VALUE -> sv.getF32Value();
4444
case F64_VALUE -> sv.getF64Value();
4545
case BOOL_VALUE -> sv.getBoolValue();
46+
case STRING_VALUE -> sv.getStringValue();
47+
case BYTES_VALUE -> sv.getBytesValue().toStringUtf8();
4648
default -> null;
4749
};
4850
} catch (com.google.protobuf.InvalidProtocolBufferException e) {

0 commit comments

Comments
 (0)