[tarantool-patches] Re: [PATCH] Set format for spaces with sysview engine

Kirill Yukhin kyukhin at tarantool.org
Thu Apr 18 15:08:40 MSK 2019


Hello,

On 18 апр 14:39, Vladislav Shpilevoy wrote:
> 
> 
> On 18/04/2019 14:14, Kirill Yukhin wrote:
> > Hello,
> > 
> > On 18 апр 13:43, Vladislav Shpilevoy wrote:
> >> Hi! Thanks for the fixes! See 4 comments below.
> >>
> >> 1. Now I see that sysview tuple format leaks. Please,
> >> apply this diff:
> >>
> >> ======================================================
> >> diff --git a/src/box/sysview.c b/src/box/sysview.c
> >> index 0b07c9f4a..96c5e78ca 100644
> >> --- a/src/box/sysview.c
> >> +++ b/src/box/sysview.c
> >> @@ -545,6 +545,8 @@ sysview_engine_create_space(struct engine *engine, struct space_def *def,
> >>  		free(space);
> >>  		return NULL;
> >>  	}
> >> +	/* Format is now referenced by the space. */
> >> +	tuple_format_unref(format);
> >>  	return space;
> >>  }
> >> ======================================================
> > 
> > Done.
> > 
> > Branch force-pushed and re-tested.
> 
> How was it retested?

I didn't staged updated result. Done.

--
Regards, Kirill Yukhin




More information about the Tarantool-patches mailing list