diqiu50 commented on PR #13749: URL: https://github.com/apache/gravitino/pull/13749#issuecomment-6594708721
Thanks, my earlier points are all addressed. On @jiangxt2's epoch-boundary P2, I agree with the strict option: for `toYear`/`toYYYYMM`, only drop `toDate` when the source column is `Date`, and keep the `DateTime` case unstructured with the raw `partition-key`; the other branches can stay as is. -- 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]
