Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MapHasher: Do not return error from HashLeaf #1611

Merged
merged 1 commit into from
May 16, 2019

Conversation

pav-kv
Copy link
Contributor

@pav-kv pav-kv commented May 16, 2019

Hashes to hashes and errors to errors.

@pav-kv pav-kv requested a review from pphaneuf May 16, 2019 14:52
@codecov
Copy link

codecov bot commented May 16, 2019

Codecov Report

Merging #1611 into master will increase coverage by 0.07%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1611      +/-   ##
==========================================
+ Coverage   67.14%   67.21%   +0.07%     
==========================================
  Files         110      110              
  Lines        8963     8950      -13     
==========================================
- Hits         6018     6016       -2     
+ Misses       2341     2334       -7     
+ Partials      604      600       -4
Impacted Files Coverage Δ
server/map_rpc_server.go 29.9% <0%> (+0.28%) ⬆️
merkle/map_verifier.go 94.28% <100%> (+5.09%) ⬆️
integration/map.go 45.78% <100%> (+0.54%) ⬆️
merkle/coniks/coniks.go 81.53% <100%> (ø) ⬆️
testonly/mapcontents.go 31.57% <100%> (-0.04%) ⬇️
merkle/maphasher/maphasher.go 75.6% <100%> (+2.88%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9366bd2...de98afa. Read the comment docs.

@pav-kv pav-kv requested review from gdbelvin and removed request for pphaneuf May 16, 2019 15:48
@gdbelvin gdbelvin merged commit a077b04 into google:master May 16, 2019
@pav-kv pav-kv deleted the map_hasher_no_error branch May 16, 2019 16:14
gdbelvin added a commit to gdbelvin/keytransparency that referenced this pull request May 16, 2019
gdbelvin added a commit to google/keytransparency that referenced this pull request May 16, 2019
gdbelvin added a commit that referenced this pull request May 17, 2019
* master: (54 commits)
  Couple of changes to make NodeIDs more frugal. (#1613)
  compact.Tree: Change semantic of adding leaves (#1592)
  MapHasher: Do not return error from HashLeaf (#1611)
  Move Postgres schema into "schema" subdirectory
  Move MySQL schema into "schema" subdirectory
  Improve FlipRightBit by doing it directly. (#1610)
  log_subtree_cache: Use compact Range instead of Tree (#1609)
  LogHasher: Return no error from HashLeaf method (#1608)
  Remove update_changelog.sh
  Fewer copies of the bytes from big.Int (#1602)
  Switch some low-level logging in hashing related code to 'if' guards (#1601)
  Merge pull request #1596 from RJPercival/del_dockerfile_db
  Correct name of "licenses" binary in .gitignore
  compact.Tree: Allow nil visitor and add benchmarks (#1599)
  Add timeout to sequencing context (#1595)
  Combine `go install` commands in .travis.yml
  Combine `go get` commands in .travis.yml
  Sort `go get` commands in .travis.yml
  Rename "indirect/external.go" to "indirect/indirect.go"
  Fix typo: "TestLibaries" -> "TestLibraries"
  ...
gdbelvin added a commit that referenced this pull request May 23, 2019
* master: (94 commits)
  Complete TODO (#1632)
  fake_node_reader: Remove unused field (#1631)
  Parallelize VerifyMapLeavesResponse (#1615)
  Remove redundant root hash calculations (#1630)
  sequencer: Consolidate compact.Tree initialization (#1629)
  Remove unused NodeReader method (#1625)
  Fix bazel build (#1627)
  added/updated postgresql implementation of log_storage (#1571)
  Clean up compact.Tree tests (#1622)
  Remove old hash list format from compact.Tree (#1621)
  Mention protoc-gen-doc in README.md
  merkle: Add testonly package with golden hashes (#1620)
  compact: Simplify getting hashes in NewTreeWithState (#1618)
  compact: Implement Tree using Range (#1522)
  Guard verbose logging in merkle path code (#1604)
  Make MaskLeft of NodeID return an explicit copy. (#1612)
  Convert directly from Int.Bits() to NodeID (#1614)
  Couple of changes to make NodeIDs more frugal. (#1613)
  compact.Tree: Change semantic of adding leaves (#1592)
  MapHasher: Do not return error from HashLeaf (#1611)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants