LuciferYang opened a new pull request, #9773: URL: https://github.com/apache/paimon/pull/9773
### Purpose close #9772 `ARRAY<VECTOR(n)>` is accepted by schema validation and readable via `ColumnarArray`, but the binary data path rejected it in four places: `InternalArray.createElementGetter` (no VECTOR case — `InternalArraySerializer` failed construction), `BinaryArray.calculateFixLengthPartSize` (threw), `BinaryArray.getVector` (unconditional throw), and `InternalRow.getDataClass` (broke `copy`). This PR routes VECTOR through the existing accessors: `getVector` in the element getter, the 8-byte variable-length slot in `calculateFixLengthPartSize` (mirroring ARRAY), `readVectorData` in `BinaryArray.getVector` (mirroring `BinaryRow.getVector`), and `InternalVector` in `getDataClass`. The write side (`BinaryWriter.writeVector` + `InternalVectorSerializer`) already existed. ### Tests New `InternalArrayVectorGetterTest`: serializer construction over VECTOR, element getter read, null element through the nullable wrapper, and a full serialize→deserialize→getVector round-trip. RED verified on master (construction threw `type VECTOR not support`). ### API and Format No format change: vectors in binary rows/arrays use the existing var-length offset-and-size layout written by `writeVectorToVarLenPart`; this PR adds the missing read/validate paths. ### Documentation None. -- 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]
