Re: [Tarantool-patches] [PATCH v10 4/4] test: add replication/applier-rollback

[Cyrill Gorcunov <gorcunov@gmail.com>]

On Sat, Feb 15, 2020 at 08:38:51PM +0300, Konstantin Osipov wrote:
> * Cyrill Gorcunov <gorcunov@gmail.com> [20/02/14 17:06]:
> >  STRS(applier_state, applier_STATE);
> >  
> > @@ -832,6 +833,17 @@ applier_apply_tx(struct stailq *rows)
> >  	trigger_create(on_commit, applier_txn_commit_cb, NULL, NULL);
> >  	txn_on_commit(txn, on_commit);
> >  
> > +	/*
> > +	 * FIXME: Move this injection somewhere inside
> > +	 * txn_write, but since it has own bug (gh 4776)
> > +	 * we will cure it a bit later.
> > +	 */
> > +	ERROR_INJECT(ERRINJ_REPLICA_TXN_WRITE, {
> > +		diag_set(ClientError, ER_INJECTION,
> > +			 "replica txn write injection");
> > +		goto rollback;
> > +	});
> 
> I thought yo wrote in the changes in v10 that you pushed the error
> down into txn layer. But here you don't do it. What am I missing?

Not yet. Pushing it sown cause https://github.com/tarantool/tarantool/issues/4776
I plan to fix it and then get back to this issue. Look this
all code is still a bit vague for me, so I've to understand every
byte before doing some more deep changes. For exactly this reason
the patch in the series does NOT change the current logic of
code flow much but tries to leave everything as it were except
a hot fix for nil dereference.

That said, Kostya, I remember about your proposal and FIXME
left here for exactly that.