Skip to content

support clean metrics of OrphanFilesCleaner#117

Closed
chongchongxiao wants to merge 2 commits into
alibaba:mainfrom
chongchongxiao:add_clean_metric
Closed

support clean metrics of OrphanFilesCleaner#117
chongchongxiao wants to merge 2 commits into
alibaba:mainfrom
chongchongxiao:add_clean_metric

Conversation

@chongchongxiao

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #114

Support clean metrics of OrphanFilesCleaner

    listDirectoriesDuration
    listDirectories
    listFileStatusDuration
    listFileStatusTasks
    listUsedFilesDuration
    usedFiles
    scanOrphanFilesDuration
    orphanFiles

Tests

No.

API and Format

No.

Documentation

No.

@@ -0,0 +1,46 @@
/*
* Copyright 2024-present Alibaba Inc.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyright 2026

}

private:
std::chrono::high_resolution_clock::time_point start;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

private member use: start_

@@ -0,0 +1,46 @@
/*

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you change other file's duration_cast call to Duration class?

#include "paimon/core/manifest/manifest_file.h"
#include "paimon/core/manifest/manifest_file_meta.h"
#include "paimon/core/manifest/manifest_list.h"
#include "paimon/core/operation/metrics/clean_metrics.h"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file does not exist.

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.

[Feature] Support clean metrics

2 participants