Re: [PR] [WIP] [Arrow] Adjust ArrowFieldWriterFactory to support nullable with correct parameters [paimon]

["leaves12138 (via GitHub)" <gi...@apache.org>]

leaves12138 commented on code in PR #5522:
URL: https://github.com/apache/paimon/pull/5522#discussion_r2056054788


##########
paimon-arrow/src/main/java/org/apache/paimon/arrow/writer/ArrowFieldWriter.java:
##########
@@ -67,6 +69,9 @@ protected abstract void doWrite(
     /** Get the value from the row at the given position and write to specified row index. */
     public void write(int rowIndex, DataGetters getters, int pos) {
         if (getters.isNullAt(pos)) {
+            if (!isNullable) {
+                throw new RuntimeException();

Review Comment:
    format("ArrowFieldWriter does not support null values in non-nullable fields. Field name : %s expected not null but found null", fieldVector.getName())



-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org