thswlsqls opened a new pull request, #9864:
URL: https://github.com/apache/paimon/pull/9864
### Purpose
fix #9861
- `NestedRow.getVector` still threw `IllegalArgumentException("Unsupported
type: VectorType")`, so a primary-key table with a `ROW<..., VECTOR>` column
failed at flush: the write buffer holds `BinaryRow`, and the Parquet/Avro
writer reads the nested row via `BinaryRow.getRow()` → `NestedRow.getVector()`.
- Implement it with `MemorySegmentUtils.readVectorData`, mirroring
`BinaryRow.getVector` and `BinaryArray.getVector`; `BinaryWriter.writeVector`
already writes the nested slot, so the layout is unchanged.
- Lineage: #7204 added the stub to `BinaryArray` and `NestedRow`; #9773
fixed `BinaryArray`; this fixes the last one.
### Tests
- Added `NestedRowTest#testNestedRowWithVector`: round-trip of `ROW<INT,
ROW<INT, VECTOR(3, FLOAT)>>`, null vector field,
`InternalRow.createFieldGetter(VECTOR)`, and the multi-segment case. Fails with
the stub exception without the fix.
- `mvn -pl paimon-common clean install` (JDK 11): BUILD SUCCESS, 0 failures
(checkstyle, spotless, enforcer included); `NestedRowTest` 4/4.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
