Commits on Dec 11, 2017

1. storage: remove unnecessary sideload inlining, add assertion …

   `Replica.sendRaftMessages` used to contain logic to inline sideloaded entries
   for `MsgApps`. This was unnecessary, as `MsgApps` would never contain thin entries.
   This is because `replicaRaftStorage.Entries` already performs the sideload inlining
   for stable entries and `raft.unstable` always contain fat entries. Since these are
   the only two sources that `raft.sendAppend` gathers entries from to populate `MsgApps`,
   we should never see thin entries in `MsgApps`.
   
   This change replaces the inlining attempt with an assertion.
   
   Release note: None

   

   nvanbenschoten committed Dec 11, 2017
   Configuration menu

   • View commit details
   • Copy full SHA for cccf882
   • Browse repository at this point

   Copy the full SHA

   cccf882 View commit details

   Browse the repository at this point in the history