[Tarantool-patches] [PATCH 3/4] raft: add explicit raft argument to all functions

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Tue Nov 10 03:00:13 MSK 2020


Thanks for the review!

>> @@ -535,7 +538,6 @@ raft_write_cb(struct journal_entry *entry)
>>   static void
>>   raft_write_request(const struct raft_request *req)
>>   {
>> -    assert(raft.is_write_in_progress);
> Not related to this patch, just a thought.
> Looks like raft_write_request belongs to some other file now.
> Will you move it somewhere else?
> I don't know where to put it though. box.cc?

Yes, this is what I am going to do in the end. When I will move
raft to src/lib/raft, some box-specific functions will stay in
src/box.

The raft library will use them as virtual functions.

I will unbind raft from box in Part 2 of my work on this issue.


More information about the Tarantool-patches mailing list