cwq222 commented on PR #13749: URL: https://github.com/apache/gravitino/pull/13749#issuecomment-6598192909
> Please update the PR description to match the strict rules in `f66f7e8`: nested `toDate(col)` is unwrapped only for a `Date` source column, and `toDateTime(col)` only for `DateTime`. The motivation, user-facing example and manual validation summary still describe unwrapping `toDate(DateTime)`. Please also update the IT description to reflect its current `Date` fixture, and align all sections with the final implementation. @jiangxt2 I've updated the PR description to match the strict rules. Thanks again for the thorough review. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
