facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-10-10 00:45:18 +03:00
Code Issues Projects Releases Wiki Activity

Moved info logging to log_statistics function

Browse Source 
Summary: print_statistics and log_statistics were used in the same places in code.

Reviewed By: StanislavGlebik

Differential Revision: D17665790

fbshipit-source-id: 5d41f1b9d77edbe12db25f109a8bb171c8072fb4
This commit is contained in:
Daniel Grzegorzewski 2019-09-30 07:19:45 -07:00 committed by Facebook Github Bot
parent 71378bb0c1
commit 95ed50db45
1 changed files with 5 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
cmds/statistics_collector.rs
View File

@ -234,9 +234,11 @@ pub fn update_statistics(
.map(move |statistics| statistics)
}
pub fn print_statistics(
pub fn log_statistics(
ctx: CoreContext,
mut scuba_logger: ScubaSampleBuilder,
cs_timestamp: i64,
repo_name: String,
hg_cs_id: HgChangesetId,
statistics: RepoStatistics,
) {
@ -249,15 +251,6 @@ pub fn print_statistics(
statistics.total_file_size,
statistics.num_lines
);
}
pub fn log_statistics(
mut scuba_logger: ScubaSampleBuilder,
cs_timestamp: i64,
repo_name: String,
hg_cs_id: HgChangesetId,
statistics: RepoStatistics,
) {
scuba_logger
.add("repo_name", repo_name)
.add("num_files", statistics.num_files)
@ -322,8 +315,8 @@ fn main(fb: FacebookInit) -> Result<(), Error> {
changeset,
)
.map(move |cs_timestamp| {
print_statistics(ctx, cs_timestamp, changeset, statistics);
log_statistics(
ctx,
scuba_logger,
cs_timestamp,
repo_name,
@ -389,13 +382,8 @@ fn main(fb: FacebookInit) -> Result<(), Error> {
cur_changeset,
)
.map(move |cs_timestamp| {
print_statistics(
ctx,
cs_timestamp,
cur_changeset,
statistics,
);
log_statistics(
ctx,
scuba_logger,
cs_timestamp,
repo_name,