Details
-
Bug
-
Resolution: Unresolved
-
Major
-
7.6.0
-
Untriaged
-
0
-
Unknown
-
Plasma-April-8-2024, Plasma-April-15-2024, Plasma-April-22-2024, Plasma-April-29-2024, Plasma-May-6-2024, Plasma-May-13-2024
Description
Found during GSI test with 30% injected read/write errors.
if doCommit {
|
s.snapMutex.Lock()
|
off := int64(minLSSOffset(s.safeOffset(), s.trimOffset))
|
if s.safeSnapOffset() > LSSOffset(off) {
|
s.log.Trim(off)
|
}
|
s.log.Commit() // --- return value is not checked
|
s.lastCommitTS = time.Now()
|
s.snapMutex.Unlock()
|
}
|
Attachments
Gerrit Reviews
For Gerrit Dashboard: MB-60198 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
208713,12 | MB-60198: Simulate retryable errors in UnstableEnvironment | unstable | plasma | Status: NEW | 0 | +1 |
208749,17 | MB-60198: Handle retryable errors during lss commit | unstable | plasma | Status: NEW | 0 | +1 |
208891,13 | MB-60198: Do not retry disk I/O if recovery is cancelled | unstable | plasma | Status: NEW | 0 | -1 |
204596,13 | MB-60198 Check log.Commit() return value on lsStore.flush(...) | unstable | plasma | Status: ABANDONED | +1 | +1 |