Re: [PR] feat(spark): Implement Spark functions `url_encode`, `url_decode` and `try_url_decode` [datafusion]

["anhvdq (via GitHub)" <gi...@apache.org>]

anhvdq commented on PR #17399:
URL: https://github.com/apache/datafusion/pull/17399#issuecomment-3617542833

   Hi @Jefffrey 
   I have pushed the new changes which include:
   - More test cases as mentioned to cover the plus replacement, roundtrips, etc
   - Add a new `try_url_decode` function which is basically identical to `url_decode` except that it returns None instead of error when the value is invalid.
   
   Please help review and let me know your feedback
   Thanks


-- 
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: github-unsubscribe@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org