morningman commented on PR #69581: URL: https://github.com/apache/doris/pull/69581#issuecomment-6456686096
Addressed the fifth review round in `a5e1ead1de1`. Per finding: - 4390935904 (P2, DATE32 payloads hidden by null parents): fixed. The array, map and struct SerDes no longer decode child values hidden by a null parent row, so element checks such as the DATE32 range only see visible values. New tests: `ArrowParentNullsTest` and `LanceTableReaderTypeTest.IgnoresDate32PayloadHiddenByNullParents`. - 4373822200 (P1, endpoint identity in the data cache key): unchanged, as answered in the thread. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
