Skip to content

Fix LOAD rate limiting to account for source TsFile bytes - #19159

Closed
Caideyipi wants to merge 1 commit into
dev/1.3from
codex/backport-load-rate-limiting-19141-dev13
Closed

Caideyipi wants to merge 1 commit into
dev/1.3from
codex/backport-load-rate-limiting-19141-dev13

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Backport of apache/iotdb#19141, commit c4086c5df64dcebf052b3f08ed462ed5293ab1cb.

loadNewTsFile can move or remove the source TsFile before rate-limit accounting reads the resource path. Use the original tsFileToLoad length so the limiter charges the source bytes accurately.

The regression test covers both deleteOrigin values, verifies the recorded byte count, and checks file transfer behavior. It is adapted to the dev/1.3 TsFile path, Tablet, and four-argument loadNewTsFile APIs.

Validation: Spotless passed. Local compilation was skipped; CI will compile the change.

@sonarqubecloud

sonarqubecloud Bot commented Nov 6, 2026

Copy link
Copy Markdown

@Caideyipi Caideyipi closed this Nov 6, 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.

1 participant