URL: https://github.com/SSSD/sssd/pull/94 Title: #94: Enable {socket,dbus}-activation for responders
fidencio commented: """ On Tue, Nov 29, 2016 at 9:33 AM, Pavel Březina <notificati...@github.com> wrote: > One thing I don't like about those patches is that we always recreate the > idle timer. Can we get around this (maybe also for the client idle > timeout)? What I have i mind is this: > > 1. Instead of resetting the timer, remember time(NULL) of the last > communication, say last_request_time > 2. Create a periodic timer that fires each responder_idle_timeout / 2 > 3. In the timer check that last_request_time + responder_idle_timeout > < time(NULL) > > It won't be as precise as your solution, but I don't think we need it to > be but it will save us lots of memory operations. What do you think? > Fine by me. Your suggestion has been used in a few other places of the our codebase as well. > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/SSSD/sssd/pull/94#issuecomment-263507800>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AAG4euHebmR2oC8k9VYhGYHpTzyt5RFvks5rC-NigaJpZM4K8AJs> > . > """ See the full comment at https://github.com/SSSD/sssd/pull/94#issuecomment-263514555
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
