JingsongLi commented on PR #10885:
URL: https://github.com/apache/paimon/pull/10885#issuecomment-6557657429

   I have now reviewed the earlier 
[#10874](https://github.com/apache/paimon/pull/10874) as well. Both PRs fix 
#10873, and their complete production implementations are identical apart from 
one explanatory comment. #10874 was opened first and already covers a real 
write/commit/read plus mismatched-formatter rejection.
   
   This PR is technically correct: its 57 targeted 
write/partition-expiry/extractor tests passed under the normal JDK 8 Maven 
checks, and CI is green. However, it adds no independent production behavior 
beyond the earlier PR. Closing this duplicate so the fix stays in one place. 
Please contribute the useful append/primary-key, pre-epoch/leap-day and 
mixed-hour regression coverage to #10874 rather than maintain a second 
implementation.
   


-- 
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]

Reply via email to