steFaiz commented on PR #8354:
URL: https://github.com/apache/paimon/pull/8354#issuecomment-4797108432
Hi @Liulietong , could you please help review this revert PR?
I may be missing something, but my current understanding is that the
scenario fixed by the original PR is not
reachable under the current lookup compaction flow. A lower high-level
record, such as L1, should already be
the materialized merged state over older higher levels like L2, so I am
not sure how we can naturally get `L1
sequence=7` and `L2 sequence=8` for the same key.
The original test seems to manually construct this state instead of
producing it through normal write and
compaction logic. If this state is actually possible, could you please
share a concrete scenario based on the
current 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]
