[Tarantool-patches] [PATCH v4 07/12] txm: introduce memtx_story

Nikita Pettik korablev at tarantool.org
Wed Sep 23 13:25:36 MSK 2020


On 22 Sep 20:51, Aleksandr Lyapunov wrote:
> Thank for the review.
> Most is fixed, see my replies below.
> 
> On 15.09.2020 17:33, Nikita Pettik wrote:
> 
> > > +int
> > > +memtx_tx_track_read(struct txn *txn, struct space *space, struct tuple *tuple)
> > > +{
> > > +	if (tuple == NULL)
> > > +		return 0;
> > > +	if (txn == NULL)
> > > +		return 0;
> > > +	if (space == NULL)
> > > +		return 0;
> > Space seems to be always != NULL. If so, let's replace it with an assertion.
> I found out that in iterators the space can be NULL. I'm not sure
> why, but I happens without my patch set. I just accepted it.
>

Hm, sounds strange. I've replaced this check with an assertion and all tests
seem to pass..
 


More information about the Tarantool-patches mailing list