Page MenuHomeSolus

Update to v4.0.2
ClosedPublic

Authored by mariosant on Oct 2 2017, 8:58 PM.
Tags
None
Referenced Files
F11017237: D1136.diff
Sun, Aug 6, 12:07 AM
F11015165: D1136.id2715.diff
Sat, Aug 5, 10:06 AM
F10890118: D1136.id.diff
Jun 28 2023, 11:14 AM
F10833690: D1136.diff
Jun 7 2023, 1:09 AM
F10804352: D1136.id2791.diff
May 29 2023, 3:47 PM
F10804228: D1136.id2716.diff
May 29 2023, 3:11 PM
F10804221: D1136.id2715.diff
May 29 2023, 3:09 PM
F10802609: D1136.id2717.diff
May 29 2023, 7:54 AM
Subscribers

Details

Summary

Upgrade urgency HIGH: Several potentially critical bugs fixed.

  1. A number of bugs were fixed in the area of PSYNC2 replication in the

specific area of restarting an instance with an RDB file having the
repliacation meta-data to continue without a full resynchronization. The
old code allowed several inconsistencies under certain conditions, like
starting a master with an RDB file generated by a slave, and later using
such master to connect previous slaves having the same replication history.
Because of other bugs, sometimes the replication resulted in a full
synchronization even if actually a partial resynchronization was possible
and so forth. Several commits by different authors fix different bugs here.

  1. AOF flush on SHUTDOWN did not cared to really write the AOF buffers

(not in the kernel but in the Redis process memory) to disk before exiting.
Calling SHUTDOWN during traffic resulted into not every operation to be
persisted on disk.

  1. The SLOWLOG could reference values inside string objects stored at keys,

creating a race condition during FLUSHALL ASYNC while the DB is reclaimed
in another thread.

Test Plan

Built and installed Redis package locally. Started redis-server as non root user. Used redis-cli to connect to the running instance. Created a record. Everything seems to work fine.

Diff Detail

Repository
R2799 redis
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

JoshStrobl added a subscriber: JoshStrobl.

Needs a summarized changelog and a test plan on how you tested it.

This revision now requires changes to proceed.Oct 2 2017, 9:00 PM
mariosant edited edge metadata.
mariosant edited the summary of this revision. (Show Details)

Added summarized changelog.

mariosant edited the test plan for this revision. (Show Details)

Added test plan.

@JoshStrobl, is that acceptable now? Thanks for your time so far.

This revision is now accepted and ready to land.Oct 5 2017, 12:53 PM
This revision was automatically updated to reflect the committed changes.