GideonPotok commented on PR #45738: URL: https://github.com/apache/spark/pull/45738#issuecomment-2048941620
@uros-db No problem at all. if I understand your refactor correctly, my changes will basically either stay in the same place or move to the new `common/unsafe/src/main/java/org/apache/spark/sql/catalyst/util/CollationSupport.java`, right? I will in the meantime take care of implementing https://issues.apache.org/jira/browse/SPARK-47412 and will also then put that on hold until after the refactor is merged. If that sounds good? -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org