jackylee-ch opened a new pull request, #85:
URL: https://github.com/apache/paimon-mosaic/pull/85

   ## Purpose
   
   `StringArray::new` is `try_new().unwrap()`, so reading a Utf8 column whose 
bytes are not valid UTF-8 panics inside arrow instead of returning an error. 
Across the JNI/FFI boundary a panic aborts the host process rather than raising.
   
   ## Changes
   
   - Build the Utf8 array with `try_new` and map the failure to `InvalidData`. 
`build_array` already returns `io::Result`.
   
   ## Tests
   
   - `test_build_array_non_utf8_returns_invalid_data_without_panicking`.
   - `cargo fmt --all -- --check`, `cargo clippy --all-targets --workspace -- 
-D warnings`, `cargo test --release --workspace`: 475 passed, 0 failed (interop 
cases skipped locally).
   


-- 
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]

Reply via email to