Skip to content

[hotfix] Make KvFileHandle Serializable#17

Merged
luoyuxia merged 1 commit into
mainfrom
KvFileHandle-Serializable
Nov 27, 2024
Merged

[hotfix] Make KvFileHandle Serializable#17
luoyuxia merged 1 commit into
mainfrom
KvFileHandle-Serializable

Conversation

@lincoln-lil

Copy link
Copy Markdown
Contributor

Since the child class of KvFileHandle declares uid field, we'd better make it Serializable.

@luoyuxia luoyuxia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@luoyuxia luoyuxia merged commit 9df2189 into main Nov 27, 2024
@wuchong wuchong deleted the KvFileHandle-Serializable branch November 27, 2024 16:12
platinumhamburg added a commit to platinumhamburg/fluss that referenced this pull request Apr 8, 2026
- Remove unused recordBatchFilter field in LogFetcher (#1)
- Remove unused getTargetSchemaId() in LogRecordReadContext (#2)
- Refactor FilterContext to internally construct PredicateSchemaResolver,
  eliminating nullable resolver and simplifying callers (#3/apache#4/apache#6/apache#9)
- Remove @nullable from getSchemaGetter() in LogRecordReadContext (#3)
- Reject negative filterSchemaId in Replica and ServerRpcMessageUtils
  with fail-fast validation (apache#5/apache#17)
- Fix CommonRpcMessageUtils to preserve records when filteredEndOffset
  is set, avoiding data loss when first batch matches but trailing
  batches are filtered (apache#18)
- Remove redundant @VisibleForTesting read() overload in LogSegment
Ugbot pushed a commit to Ugbot/fluss that referenced this pull request Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants