Currently, the migrationId member variable is stored as a normal member variable of the MigrationDestinationManager class. However, it's undefined at construction time as it gets initialized with the state passed in from the recvChunkStart command. So it should be converted to a boost::optional so that it doesn't have to make the MigrationDestinationManager a friend.