Skip to content

Conversation

@adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Create ChecksumData with checksum list, instead of updating it right after creation, to avoid unnecessarily copying the list.

https://issues.apache.org/jira/browse/HDDS-2458

How was this patch tested?

Tested on 3-node cluster with a 120MB key. ozone sh key get verifies the checksum.

ozone freon ockg -p test -t 1 -n 1 # easy volume and bucket creation ;)
cat share/ozone/lib/*jar > tmp.jar
ozone sh key put vol1/bucket1/test/large tmp.jar
ozone sh key get vol1/bucket1/test/large tmp.jar.out
diff -q tmp.jar tmp.jar.out

@nandakumar131 nandakumar131 merged commit 42b6909 into apache:master Nov 13, 2019
@adoroszlai adoroszlai deleted the HDDS-2458 branch November 13, 2019 17:29
@adoroszlai
Copy link
Contributor Author

Thanks @nandakumar131 for reviewing and merging it.

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.

2 participants