Re: [Twisted-Python] List of required builds before a merge

2018-03-12 Thread Glyph
On Mar 12, 2018, at 10:27 PM, Amber Brown wrote: > > The buildbots went after someone said that the RAX hosting was going away, > and I (and a few others) didn't get the (annoyingly quiet) correction that it > was only for new projects (which was not how the original was written). All > the tw

Re: [Twisted-Python] List of required builds before a merge

2018-03-12 Thread Amber Brown
The buildbots went after someone said that the RAX hosting was going away, and I (and a few others) didn't get the (annoyingly quiet) correction that it was only for new projects (which was not how the original was written). All the twisted list got was the original letter from the SFC without a "f

Re: [Twisted-Python] List of required builds before a merge

2018-03-12 Thread Glyph
> On Mar 12, 2018, at 4:59 AM, Adi Roiban wrote: > > Hi, > > It is not clear to me what builders need to pass before we can merge > something. > > I expect that all supported "platforms" need to pass, but it is not > clear what are the currently supported platforms. > > We have this info in

Re: [Twisted-Python] Waiting for a contended resource

2018-03-12 Thread Richard van der Hoff
Thank you for all for all the answers so far, particularly to Ilya and Jean-Paul who provided some very helpful code samples. It's interesting to realise that, by avoiding locking, we can end up with a much more efficient implementation. I'll have to figure out how widely we can apply this tec

Re: [Twisted-Python] Waiting for a contended resource

2018-03-12 Thread Ilya Skriblovsky
Thanks for correction, Jean-Paul, you're absolutly right пн, 12 мар. 2018 г. в 23:00, Jean-Paul Calderone : > On Mon, Mar 12, 2018 at 3:52 PM, Ilya Skriblovsky < > ilyaskriblov...@gmail.com> wrote: > >> Hi, Richard, >> >> I've used class like this to cache the result of Expensive Calculation: >>

Re: [Twisted-Python] Waiting for a contended resource

2018-03-12 Thread Jean-Paul Calderone
On Mon, Mar 12, 2018 at 3:52 PM, Ilya Skriblovsky wrote: > Hi, Richard, > > I've used class like this to cache the result of Expensive Calculation: > > class DeferredCache: > pending = None > result = None > failure = None > > def __init__(self, expensive_func): > self.exp

Re: [Twisted-Python] Waiting for a contended resource

2018-03-12 Thread Ilya Skriblovsky
Hi, Richard, I've used class like this to cache the result of Expensive Calculation: class DeferredCache: pending = None result = None failure = None def __init__(self, expensive_func): self.expensive_func = expensive_func def __call__(self): if self.pending

Re: [Twisted-Python] Waiting for a contended resource

2018-03-12 Thread L. Daniel Burr
Hi Richard, On March 12, 2018 at 1:49:41 PM, Richard van der Hoff (rich...@matrix.org) wrote: Hi folks, I thought I'd poll the list on the best way to approach a problem in  Twisted. The background is that we have a number of resources which can be  requested by a REST client, and which are cal

[Twisted-Python] Waiting for a contended resource

2018-03-12 Thread Richard van der Hoff
Hi folks, I thought I'd poll the list on the best way to approach a problem in Twisted. The background is that we have a number of resources which can be requested by a REST client, and which are calculated on demand. The calculation is moderately expensive (can take multiple seconds), so th

Re: [Twisted-Python] List of required builds before a merge

2018-03-12 Thread Jean-Paul Calderone
On Mon, Mar 12, 2018 at 7:59 AM, Adi Roiban wrote: > Hi, > > It is not clear to me what builders need to pass before we can merge > something. > > I expect that all supported "platforms" need to pass, but it is not > clear what are the currently supported platforms. > > We have this info in the w

[Twisted-Python] List of required builds before a merge

2018-03-12 Thread Adi Roiban
Hi, It is not clear to me what builders need to pass before we can merge something. I expect that all supported "platforms" need to pass, but it is not clear what are the currently supported platforms. We have this info in the wiki but it does not help. https://twistedmatrix.com/trac/wiki/Review