databendlabs / databend

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

refactor: unify path of write segment #16517

Closed SkyFan2002 closed 1 day ago

SkyFan2002 commented 1 day ago

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

Summary

  1. remove duplicated SegmentWriter, SegmentsIO::write_segment(), use Segment::write_meta() and Segment::write_meta_through_cache() should be ok.
  2. remove impl CachedObject<CompactSegmentInfo> for CompactSegmentInfo

Tests

Type of change


This change isโ€‚Reviewable