diff options
author | Peter Dillinger <peterd@fb.com> | 2020-05-21 08:10:43 -0700 |
---|---|---|
committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | 2020-05-21 08:12:51 -0700 |
commit | c7aedf1b48d8dfd9a2726b20bec9daee7be833f4 (patch) | |
tree | 1726a382135032b70a9d78a62ed4f8dccbaf65b9 /.github | |
parent | eb04bb86c6e713230ab07ee7ab919feeb9f891e0 (diff) |
Clean up some code related to file checksums (#6861)
Summary:
* Add missing unit test for schema stability of FileChecksumGenCrc32c
(previously was only comparing to itself)
* A lot of clarifying comments
* Add some assertions for preconditions
* Rename WritableFileWriter::CalculateFileChecksum -> UpdateFileChecksum
* Simplify FileChecksumGenCrc32c with shared functions
* Implement EndianSwapValue to replace unused EndianTransform
And incidentally since I had trouble with 'make check-format' GitHub action disagreeing with local run,
* Output full diagnostic information when 'make check-format' fails in CI
Pull Request resolved: https://github.com/facebook/rocksdb/pull/6861
Test Plan: new unit test passes before & after other changes
Reviewed By: zhichao-cao
Differential Revision: D21667115
Pulled By: pdillinger
fbshipit-source-id: 6a99970f87605aa024fa540c78cd519ff322c3e6
Diffstat (limited to '.github')
-rw-r--r-- | .github/workflows/sanity_check.yml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/.github/workflows/sanity_check.yml b/.github/workflows/sanity_check.yml index 92b25bb7b..3ee92fc85 100644 --- a/.github/workflows/sanity_check.yml +++ b/.github/workflows/sanity_check.yml @@ -35,7 +35,7 @@ jobs: args: https://raw.githubusercontent.com/llvm-mirror/clang/master/tools/clang-format/clang-format-diff.py - name: Check format - run: make check-format + run: VERBOSE_CHECK=1 make check-format - name: Compare buckify output run: make check-buck-targets |