[PATCH] Multi-process perf test
Douglas Bagnall
douglas.bagnall at catalyst.net.nz
Thu Jun 29 01:56:41 UTC 2017
On 29/06/17 12:30, Andrew Bartlett wrote:
> On Thu, 2017-06-29 at 12:00 +1200, Douglas Bagnall wrote:
>> It seems the extra imports in the new files were cut-and-paste
>> inherited from ad_dc_multi_bind.py, so I have taken the liberty of
>> fixing them there and squashed it all into the attached patch. It
>> appears to run.
>>
>> Also, line length and whitespace are close to perfect!
>>
>
> Thanks Douglas.
>
> Regarding the ANCIENT_SAMBA stuff, given we require working python
> messaging, unless we backport that or it is reworked to just use
> os.pipe() it isn't going to be even on Samba 4.6, let alone < 4.3.
>
> Should we keep that just in case we backport, or just drop it?
Well, I doubt we'll backport it to 4.2, which is where ANCIENT_SAMBA
applies. If someone (like, probably you) wants to run these tests on
those versions they can backport as necessary in their own branches.
If you aren't planning such a backport, there is a very high chance
that the ANCIENT_SAMBA sections will never be run and thus should be
removed.
Douglas
>
> Andrew Bartlett
>
> --
> Andrew Bartlett http://samba.org/~abartlet/
> Authentication Developer, Samba Team http://samba.org
> Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
>
More information about the samba-technical
mailing list