HyukjinKwon opened a new pull request, #37067:
URL: https://github.com/apache/spark/pull/37067

   ### What changes were proposed in this pull request?
   
   This PR is sort of a followup of 
https://github.com/apache/spark/commit/88d8de9260edf6e9d5449ff7ef6e35d16051fc9f 
to respect interpreted projections in `RowEncoder`. Similar fix was made before 
in deserialization side at 
https://github.com/apache/spark/commit/26128484228089c74517cd15cef0bb4166a4186f,
 and this PR completes it in serialization side too.
   
   ### Why are the changes needed?
   
   This is a symmetry with 
https://github.com/apache/spark/commit/26128484228089c74517cd15cef0bb4166a4186f.
 We should make the configurations respected consistently.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, by default.
   Yes, when users set `spark.sql.codegen.factoryMode` to other values. The 
operations such as DSv1, DSv2, or object conversions such as `MapObjects` will 
respect the value.
   
   ### How was this patch tested?
   
   Existing test cases at `RowEncoderSuite extends CodegenInterpretedPlanTest` 
should cover 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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to