Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Fix hadoop-ai ecc eccor detection #2343

Merged
merged 3 commits into from
Mar 18, 2019
Merged

Fix hadoop-ai ecc eccor detection #2343

merged 3 commits into from
Mar 18, 2019

Conversation

mzmssg
Copy link
Member

@mzmssg mzmssg commented Mar 18, 2019 •
edited
Loading

Origin code only consider ecc code 0/1, but it's actually an int.

Readable patch is here:
mzmssg/hadoop#3

@mzmssg mzmssg requested a review from fanyangCS March 18, 2019 11:12
@mzmssg
Copy link
Member Author

mzmssg commented Mar 18, 2019

#2146

@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.647% when pulling 8b84b11 on zimiao/ecc_error into a8db66e on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.647% when pulling 8b84b11 on zimiao/ecc_error into a8db66e on master.

@fanyangCS fanyangCS requested a review from yqwang-ms March 18, 2019 11:32
@mzmssg mzmssg merged commit b218973 into master Mar 18, 2019
@mzmssg mzmssg mentioned this pull request Mar 20, 2019
Closed
@mzmssg mzmssg deleted the zimiao/ecc_error branch March 28, 2019 08:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@yqwang-ms yqwang-ms yqwang-ms approved these changes

@fanyangCS fanyangCS Awaiting requested review from fanyangCS

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mzmssg @coveralls @yqwang-ms