riak_repl2_fssource and riak_repl2_fssink have a bunch of case statements based on Strategy which is either keylist or aae. These case statements should be replaced with a utility function that takes in Strategy and sink or source and returns the necessary module. (ex. riak_repl_aae_source or riak_repl_keylist_server. This would be much cleaner than the current implementation especially for cancel_fullsync calls and other similar handles.
riak_repl2_fssourceandriak_repl2_fssinkhave a bunch ofcasestatements based onStrategywhich is eitherkeylistoraae. These case statements should be replaced with a utility function that takes inStrategyandsinkorsourceand returns the necessary module. (ex.riak_repl_aae_sourceorriak_repl_keylist_server. This would be much cleaner than the current implementation especially forcancel_fullsynccalls and other similar handles.