Skip to content

fix(db): reserve putting byte[0] in db recovery#4703

Merged
tomatoishealthy merged 1 commit intotronprotocol:release_v4.6.0from
tomatoishealthy:fix/db-recover-wrong-deletion
Oct 9, 2022
Merged

fix(db): reserve putting byte[0] in db recovery#4703
tomatoishealthy merged 1 commit intotronprotocol:release_v4.6.0from
tomatoishealthy:fix/db-recover-wrong-deletion

Conversation

@tomatoishealthy
Copy link
Copy Markdown
Contributor

@tomatoishealthy tomatoishealthy commented Oct 9, 2022

What does this PR do?

The operation of putting byte[0] should be reserved instead of being deleted.

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 9, 2022

Codecov Report

❗ No coverage uploaded for pull request base (release_v4.6.0@d432b5d). Click here to learn what that means.
The diff coverage is n/a.

@@                Coverage Diff                @@
##             release_v4.6.0    #4703   +/-   ##
=================================================
  Coverage                  ?   54.19%           
  Complexity                ?     8002           
=================================================
  Files                     ?      832           
  Lines                     ?    47417           
  Branches                  ?     5149           
=================================================
  Hits                      ?    25699           
  Misses                    ?    19677           
  Partials                  ?     2041           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tomatoishealthy tomatoishealthy changed the title fix(db): reserve the operation of putting byte[0] in db recovery fix(db): reserve putting byte[0] in db recovery Oct 9, 2022
@tomatoishealthy tomatoishealthy merged commit 161e4ca into tronprotocol:release_v4.6.0 Oct 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants