You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/10/19 15:51:56 UTC

[GitHub] [spark] mkaravel commented on pull request #34154: [SPARK-37047][SQL] Add lpad and rpad functions for binary strings

mkaravel commented on pull request #34154:
URL: https://github.com/apache/spark/pull/34154#issuecomment-946861549


   > What's the use case for this? given that we would have to justify a breaking change (of behavior that was probably not intended to work anyway though)
   
   From my point of view, the old behavior is wrong and would have to be fixed anyways. Besides this, the main scenario is one where a user has a BINARY column with different lengths and they want to "align" all values either to the left or to the right. These overloads allow the user to do that.


-- 
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: reviews-unsubscribe@spark.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org