[GitHub] [iceberg] RussellSpitzer commented on pull request #8192: Spark: Correct partition transform functions to match spec.md

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

RussellSpitzer commented on PR #8192:
URL: https://github.com/apache/iceberg/pull/8192#issuecomment-1663019099

   I would want @rdblue to check this. If I remember correctly we originally did this because the base implementation happened to be wrong so we had to add another implementation that took account for 0 correctly. 
   
   The other issue here is that the non-plural names are Spark native functions, is that an issue?


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org