boluor commented on issue #3086: URL: https://github.com/apache/doris-website/issues/3086#issuecomment-4525904472
Thanks for the structured feedback — the Compaction docs do have room to improve. Taking them in order: 1. **Manually trigger base vs cumulative** — engine accepts \`POST /api/compaction/run?tablet_id=...&compact_type=cumulative|base\`. \`full\` is not exposed via the BE HTTP API; the closest thing is base compaction over a tablet whose rowset chain reaches the earliest version. The doc should call this out. 2. **Does base compaction merge everything?** — base compaction merges every rowset older than the current "base" pointer into a single rowset; if no fresh cumulative segments exist on top, the result is effectively "all rowsets so far". The wording in the doc undersells that. 3. **Score ≤ 5 ceiling** — when auto-compaction has stopped, manual base compaction can still be triggered for the cases above; that should be in the FAQ. Plus your layout nits: - Search ranking for "Compaction" landing on the BE REST API page rather than the concept page — that's an Algolia config concern; will raise with the docs infra team. - Adding a note on import + query latency under heavy ingest is a fair add to the load-manual. - Cross-link \`COMPACT TABLE\` → Compaction chapter — straightforward edit. Will batch these into a single Compaction docs follow-up. Leaving this issue open until the PR lands. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
