[tarantool-patches] Re: [server-dev] [PATCH] lua: pwd: split data fetch from deserialization

Kirill Yukhin kyukhin at tarantool.org
Fri Sep 13 11:15:52 MSK 2019


Hello,

On 05 Sep 00:02, Vladislav Shpilevoy wrote:
> Hi! Thanks for the patch!
> 
> LGTM.
> 
> On 04/09/2019 00:42, Alexander Turenko wrote:
> > This commit does not change a user visible behaviour. It refactors pwd
> > module to explicitly divide code that fetches data from passwd / group
> > databases from code that performs deserialization of the data to Lua
> > tables.
> > 
> > The idea of splitting of those actions appears when it was observed that
> > a call of getpw() / getgr() leads to problems on some systems when it is
> > invoked during passwd / group database traveral.
> > 
> > Now it is more obvious that we don't call getpw() during passwd
> > traversal and getgr() during group traveral.
> > 
> > Follows up #4428 and #4447.

I've checked your patch into 1.10, 2.1, 2.2 and master.

--
Regards, Kirill Yukhin




More information about the Tarantool-patches mailing list