diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2021-12-08 14:01:54 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2021-12-08 14:01:54 +0000 |
commit | 65c857688005ac042809d034a27c9eab586eb4d3 (patch) | |
tree | 351113d0c70161484d170e74fe2b6abfacccc3b6 /searchcore | |
parent | bf37710f4ce4dd8e676e19e102e77272237d8b52 (diff) |
- Add class comments.
- Unify info logging.
- Demote some info logging to debug.
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/vespa/searchcore/proton/docsummary/summarycompacttarget.h | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/searchcore/src/vespa/searchcore/proton/docsummary/summarycompacttarget.h b/searchcore/src/vespa/searchcore/proton/docsummary/summarycompacttarget.h index 529aa3b816e..083f763d8e6 100644 --- a/searchcore/src/vespa/searchcore/proton/docsummary/summarycompacttarget.h +++ b/searchcore/src/vespa/searchcore/proton/docsummary/summarycompacttarget.h @@ -32,11 +32,14 @@ private: virtual size_t getBloat(const IDocumentStore & docStore) const = 0; virtual Task::UP create(IDocumentStore & docStore, FlushStats & stats, SerialNum currSerial) = 0; - vespalib::Executor &_summaryService; - IDocumentStore & _docStore; - FlushStats _lastStats; + vespalib::Executor &_summaryService; + IDocumentStore & _docStore; + FlushStats _lastStats; }; +/** + * Implements target to compact away removed documents. Wasted disk space is cost factor used for prioritizing. + */ class SummaryCompactBloatTarget : public SummaryGCTarget { private: size_t getBloat(const search::IDocumentStore & docStore) const override; @@ -45,6 +48,10 @@ public: SummaryCompactBloatTarget(vespalib::Executor & summaryService, IDocumentStore & docStore); }; +/** + * Target to ensure bucket spread is kept low. The cost is reported as a potential gain in disk space as + * we do not have a concept for bucket spread. + */ class SummaryCompactSpreadTarget : public SummaryGCTarget { private: size_t getBloat(const search::IDocumentStore & docStore) const override; |