Fokko commented on issue #10616: URL: https://github.com/apache/iceberg/issues/10616#issuecomment-2200191427
Hey @lurnagao-dahua Thanks for raising this, and @ajantha-bhat for jumping in here. This was discussed earlier in https://github.com/apache/iceberg/issues/10159 and points to a comment in https://github.com/apache/iceberg/issues/279#issuecomment-519620975. Some historical context: - `int -> date` in https://github.com/apache/iceberg/pull/447/ - `date -> int` in https://github.com/apache/iceberg/pull/5980/ - Discussed earlier this year in https://github.com/apache/iceberg/pull/9345/ So the logical type is a `date`, but the physical type is still an `int32`, maybe we should make that more explicit? It seems to cause a lot of confusion. Looking forward to hearing what others think. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
