Skip to content

Commit 92d5041

Browse files
gabeleviGabe Levi
authored andcommitted
Add data to recheck logging
Reviewed By: gkz Differential Revision: D7494632 fbshipit-source-id: 78700ae384e18aad262153ceb34106aaab87d099
1 parent 1d27f14 commit 92d5041

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

src/services/inference/types_js.ml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -772,7 +772,6 @@ let recheck_with_profiling ~profiling ~options ~workers ~updates env ~force_focu
772772
let%lwt new_or_changed, freshparsed, unparsed, new_local_errors =
773773
reparse ~options ~profiling ~workers modified in
774774
let freshparsed = freshparsed |> FilenameMap.keys |> FilenameSet.of_list in
775-
let new_or_changed_count = FilenameSet.cardinal new_or_changed in
776775

777776
(* clear errors for new, changed and deleted files *)
778777
let errors =
@@ -1105,21 +1104,21 @@ let recheck_with_profiling ~profiling ~options ~workers ~updates env ~force_focu
11051104
checked_files = checked;
11061105
errors;
11071106
},
1108-
(new_or_changed_count, deleted_count, !dependent_file_count))
1107+
(new_or_changed, deleted, all_dependent_files))
11091108
)
11101109

11111110
let recheck ~options ~workers ~updates env ~force_focus =
11121111
let should_print_summary = Options.should_profile options in
1113-
let%lwt profiling, (env, (modified_count, deleted_count, dependent_file_count)) =
1112+
let%lwt profiling, (env, (modified, deleted, dependent_files)) =
11141113
Profiling_js.with_profiling_lwt ~should_print_summary (fun profiling ->
11151114
recheck_with_profiling ~profiling ~options ~workers ~updates env ~force_focus
11161115
)
11171116
in
11181117
FlowEventLogger.recheck
11191118
(** TODO: update log to reflect current terminology **)
1120-
~modified_count
1121-
~deleted_count
1122-
~dependent_file_count
1119+
~modified
1120+
~deleted
1121+
~dependent_files
11231122
~profiling;
11241123
Lwt.return (profiling, env)
11251124

src/stubs/flowEventLogger.ml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@ let out_of_date _ = ()
3535
let exit _ _ = ()
3636
let report_from_monitor_server_exit_due_to_signal _ = ()
3737
let recheck
38-
~modified_count:_
39-
~deleted_count:_
40-
~dependent_file_count:_
38+
~modified:_
39+
~deleted:_
40+
~dependent_files:_
4141
~profiling:_ = ()
4242
let murdered_by_oom_killer _ = ()
4343
let ephemeral_command_success ?json_data:_ ~client_context:_ ~profiling:_ = ()

0 commit comments

Comments
 (0)