Skip to content

Conversation

@LiebingYu
Copy link
Contributor

Purpose

Currently, the value of log.segment.file-size is configured by MemorySize which could be vary large. But in fluss sever we use Java int to represent log.segment.file-size. Actually the max value of log.segment.file-size is 2GB -1 (Integer.MAX_VALUE bytes). I think we need to valid this value and fail fast if it has been set too large.

Brief change log

Tests

API and Format

Documentation

@wuchong wuchong merged commit d62e63c into apache:main Aug 10, 2025
4 of 5 checks passed
platinumhamburg pushed a commit to platinumhamburg/fluss that referenced this pull request Aug 11, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
zcoo pushed a commit to luoyuxia/fluss that referenced this pull request Aug 18, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
polyzos pushed a commit to polyzos/fluss that referenced this pull request Aug 30, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
polyzos pushed a commit to Alibaba-HZY/fluss that referenced this pull request Aug 31, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
polyzos pushed a commit to polyzos/fluss that referenced this pull request Sep 21, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
polyzos pushed a commit to polyzos/fluss that referenced this pull request Sep 21, 2025
Co-authored-by: Liebing <yuliebing.ylb@alibaba-inc.com>
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