[GitHub] [carbondata] akashrn5 commented on pull request #4151: [CARBONDATA-4211] Fix - from xx Insert into select fails if an SQL statement contains multiple inserts

[GitBox <gi...@apache.org>]

akashrn5 commented on pull request #4151:
URL: https://github.com/apache/carbondata/pull/4151#issuecomment-862135644


   Since our command classes extends spark command classes, `ExecutedCommandExec` will be the physical operator that executes the run method of a `RunnableCommand`. So we in this case we return row object, and this will be converted to `GenericInternalRow` in `sideEffectResult` of `ExecutedCommandExec`, but during Union operatio execution it expects to convert to Unsafe row and it fails with cast exception.
   
   So as suggested by spark, we can override `sideEffectResult` and handle this case with the output of our command classes, like insert into command in this case, so that we can send proper results from physical node.


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

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