luoyuxia opened a new pull request, #3506:
URL: https://github.com/apache/fluss/pull/3506

   ## Summary
   - `#3503` added `splitIndex` and `tieringRoundTimestamp` to `TieringSplit`'s 
`equals`/`hashCode`, which broke `TieringSourceEnumeratorTest`: the expected 
splits built in tests carry `UNKNOWN` round metadata while the actual assigned 
splits carry generation-time values, so `containsExactlyInAnyOrderElementsOf` 
no longer matches.
   - Refactor the test helpers: validate the per-round metadata (split indexes 
cover `0..n-1`, a single first split, a shared positive timestamp) separately 
via `assertValidTieringRound`, then normalize the volatile fields away before 
comparing the remaining fields with the regular `equals`.
   - This replaces the previous 
`assertTieringRoundMetadata`/`clearTieringSplitMetadata` helpers that mutated 
the actual splits in place, and applies the round-metadata validation 
consistently across all assignment assertions.
   - Also drop the now-unused `splitIndex`/`tieringRoundTimestamp` validation 
from `TieringSplit` and trim the redundant identity/default cases in 
`TieringSplitSerializerTest`.
   
   ## Test Plan
   - `./mvnw test -pl fluss-flink/fluss-flink-common 
-Dtest=TieringSourceEnumeratorTest,TieringSplitSerializerTest` — all pass.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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