Yicong-Huang opened a new pull request, #53368:
URL: https://github.com/apache/spark/pull/53368

   ### What changes were proposed in this pull request?
   
   Remove redundant `self._timezone` and `self._safecheck` assignments in three 
serializer classes (`ArrowStreamAggArrowUDFSerializer`, 
`ArrowStreamAggArrowIterUDFSerializer`, `ArrowStreamAggPandasUDFSerializer`).
   
   Also improve parameter passing by directly passing `assign_cols_by_name` and 
`arrow_cast` to parent class instead of passing fixed values and then 
overriding them.
   
   ### Why are the changes needed?
   
   Improves code clarity and consistency by removing redundant code that was 
already set by parent classes.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Ran existing tests.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No.
   


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