Re: [PR] [SPARK-47421][SQL] Add collation support for URL expressions [spark]

2024-05-09 Thread via GitHub


cloud-fan closed pull request #46460: [SPARK-47421][SQL] Add collation support 
for URL expressions
URL: https://github.com/apache/spark/pull/46460


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



Re: [PR] [SPARK-47421][SQL] Add collation support for URL expressions [spark]

2024-05-09 Thread via GitHub


cloud-fan commented on PR #46460:
URL: https://github.com/apache/spark/pull/46460#issuecomment-2102171977

   thanks, merging to master!


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



Re: [PR] [SPARK-47421][SQL] Add collation support for URL expressions [spark]

2024-05-08 Thread via GitHub


uros-db commented on PR #46460:
URL: https://github.com/apache/spark/pull/46460#issuecomment-2101119735

   @dongjoon-hyun please review this one


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



Re: [PR] [SPARK-47421][SQL] Add collation support for URL expressions [spark]

2024-05-08 Thread via GitHub


uros-db commented on PR #46460:
URL: https://github.com/apache/spark/pull/46460#issuecomment-2100872209

   note: collation awareness for these pass-through Spark expressions required 
modifying query plans in `query-tests/explain-results/…` in order to 
accommodate using `StringTypeAnyCollation` instead of `StringType`


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