ahsanabbas123 commented on issue #48761:
URL: https://github.com/apache/arrow/issues/48761#issuecomment-3719095757

   I suspect this could be due to serializing args here (which adds the URIs 
once): 
https://github.com/apache/arrow/blob/28855e0be2a5c43f4524e1130549c577f40da6da/cpp/src/arrow/engine/substrait/expression_internal.cc#L1401
 
   
   and then again in `EncodeSubstraitCall` (adds URIs again): 
https://github.com/apache/arrow/blob/28855e0be2a5c43f4524e1130549c577f40da6da/cpp/src/arrow/engine/substrait/expression_internal.cc#L1267
   
   Would it make sense to do the remove the former and shift it inside each 
case `struct_field` / `list_element` / `if_else` / `is_in`  as a fix for this?  
 


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