Skip to content

Conversation

@captainzmc
Copy link
Member

What changes were proposed in this pull request?

This bug should be caused by HDDS-5742. We need to pass in a ReadOnlyBuffer, otherwise there will be problems with the written data. An error is reported while reading.
In addition, we did not detect this problem because checksum was turned off by default in the previous UT.

image

What is the link to the Apache JIRA

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

How was this patch tested?

Fixed UT

@captainzmc captainzmc requested a review from szetszwo October 11, 2021 11:40
@bshashikant bshashikant merged commit 3cb1725 into apache:HDDS-4454 Oct 12, 2021
@bshashikant
Copy link
Contributor

Thanks @captainzmc for the quick fix.

Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 the change looks good.

szetszwo pushed a commit to szetszwo/ozone that referenced this pull request May 6, 2022
captainzmc added a commit to captainzmc/hadoop-ozone that referenced this pull request Jul 4, 2022
szetszwo pushed a commit that referenced this pull request Oct 25, 2022
…enabled (#2729)

(cherry picked from commit aea1d84)
(cherry picked from commit 79507fe72f98a0f02a0cad88066437fbb7d63bad)
szetszwo pushed a commit that referenced this pull request Nov 7, 2022
…enabled (#2729)

(cherry picked from commit aea1d84)
(cherry picked from commit 79507fe72f98a0f02a0cad88066437fbb7d63bad)
(cherry picked from commit b1f06e8)
szetszwo pushed a commit that referenced this pull request Dec 1, 2022
szetszwo pushed a commit that referenced this pull request Dec 16, 2022
nishitpatira pushed a commit to nishitpatira/ozone that referenced this pull request Dec 16, 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