sundapeng commented on PR #8862: URL: https://github.com/apache/paimon/pull/8862#issuecomment-5093143191
Folded into #8861. Reviewing this on its own surfaced why the split was wrong: keeping a non-empty `_temporary` alive is only safe once the read path stops treating what is inside it as data. On master that path still filters by leaf file name, so this change alone would let another job's uncommitted files reach query results — and duplicate those rows once that job commits. The two halves are one invariant (a staging directory belongs to whoever is staging there), and Apache PRs have no enforced merge order, so they now ship as one. -- 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]
