yjshen commented on PR #3354: URL: https://github.com/apache/arrow-datafusion/pull/3354#issuecomment-1237671011
Currently, the use of external `read_field_x` functions is implemented as `b.call_stmt`s, which are executed for side effects. You need to create a local variable and assign the `Expr::Call` result to it. For the reader case we are meeting here, I personally think it's OK to panic here since we are deserializing RecordBatch from Vec<u8> with a known row type. It must be a bug if we create a list of array builders according to the schema but cannot cast them to the corresponding type-specific builders later. -- 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]
