[tarantool-patches] Re: [PATCH v3 6/7] Add merger for tuples streams (C part)

Alexander Turenko alexander.turenko at tarantool.org
Thu Apr 25 18:32:11 MSK 2019


On Thu, Apr 25, 2019 at 04:45:39PM +0300, Konstantin Osipov wrote:
> * Alexander Turenko <alexander.turenko at tarantool.org> [19/04/25 16:37]:
> > merge source or merger's source. I found it neat to have one prefix
> > 'merger_' for all related structures and functions.
> 
> What naming scheme other libraries for map/reduce use?

Our merger is more like a specific reduce phase when we see on it in
context of a general map-reduce framework. It is hard to find an
intersection of our names with, say, hadoop.

Once in the past we discussed how to better integrate general map-reduce
API into net.box, but then decided to implement merger first as the
concrete reduce phase w/o attempts to generalize it much.

Anyway, I see the word 'input' is quite often in map-reduce libraries
documentation. Should we use it instead of 'source'? merge_input or
merger_input?

WBR, Alexander Turenko.



More information about the Tarantool-patches mailing list