URL: https://github.com/SSSD/sssd/pull/564
Title: #564: Do not keep allocating external groups on a long-lived context

jhrozek commented:
"""


> On 3 May 2018, at 09:07, fidencio <notificati...@github.com> wrote:
> 
> Patch looks good. I'm running the internal CI just for the sake of the 
> process and I'll add the accepted label as soon as I get the results.

btw if you are also going to push the patches, please add Sumit as the reviewer 
as well. He actually found the bug, I’m just the one who typed in the code, 
then Sumit reviewed it on IRC. I was even considering adding him as author, but 
that wouldn’t be ‘clean’ either and I hope it’s OK to commit like this.

> 
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
> 


"""

See the full comment at 
https://github.com/SSSD/sssd/pull/564#issuecomment-386213826
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to