databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.87k stars 751 forks source link

fix(storge): write progress in compact hook #16901

Closed zhyass closed 18 hours ago

zhyass commented 20 hours ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

This PR fixes an issue where the write progress could become inconsistent when compaction was triggered after a write. Previously, compaction failures would cause an early return, skipping the reset of write progress and resulting in incorrect output. After the fix, the write progress will consistently reflect the result of the write operation, regardless of whether the compaction succeeds or fails.

Tests

Type of change


This change isโ€‚Reviewable