Skip to content

Commit

Permalink
Skip mount stat logging if mounted content names are empty
Browse files Browse the repository at this point in the history
Summary:
Stats aren't super helpful without names, so let's exclude
all events without for now.

Reviewed By: marco-cova

Differential Revision: D10085437

fbshipit-source-id: a071ab6b5d02340656506ed790bfe7ebcde84c42
  • Loading branch information
passy authored and facebook-github-bot committed Sep 28, 2018
1 parent 576acd2 commit 0dd4dc7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ private void logMountPerfEvent(ComponentsLogger logger, @Nullable PerfEvent moun

// MOUNT events that don't mount any content are not valuable enough to log at the moment.
// We will likely enable them again in the future. T31729233
if (mMountStats.mountedCount == 0) {
if (mMountStats.mountedCount == 0 || mMountStats.mountedNames.isEmpty()) {
logger.cancelPerfEvent(mountPerfEvent);
return;
}
Expand Down

0 comments on commit 0dd4dc7

Please sign in to comment.