Skip to content

Commit

Permalink
Fix the vacuum log message in DRY RUN
Browse files Browse the repository at this point in the history
Currently, we output a log line with `DeltaVacuumStats` like this
```
Found 12 files and directories in a total of 1 directories that are safe to delete.DeltaVacuumStats(true,Some(0),604800000,1669825818524,1,12,10404,11374,0)
```

This change fixes it in a better format:
```
Found 12 files (10404 bytes) and directories in a total of 1 directories that are safe to delete.
```

A simple log change.

GitOrigin-RevId: 182e7c094647f639e925d714f60f2b4dc8c32944
  • Loading branch information
zsxwing authored and vkorukanti committed Dec 9, 2022
1 parent 4cd7f75 commit 94a93e6
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -241,8 +241,8 @@ object VacuumCommand extends VacuumCommandImpl with Serializable {
timeTakenForDelete = 0L)

recordDeltaEvent(deltaLog, "delta.gc.stats", data = stats)
logConsole(s"Found $numFiles files and directories in a total of " +
s"$dirCounts directories that are safe to delete.$stats")
logConsole(s"Found $numFiles files ($sizeOfDataToDelete bytes) and directories in " +
s"a total of $dirCounts directories that are safe to delete.")

return diffFiles.map(f => stringToPath(f).toString).toDF("path")
}
Expand Down

0 comments on commit 94a93e6

Please sign in to comment.