and where I captured the _persistence_id and returned it in the response


On Mon, Jan 14, 2013 at 11:41 AM, John Lockwood <
john.lockwood@webfilings.com> wrote:

> really, the changes in example were just me playing around with furious,
> making sure I understood what it was doing, except for the part of
> the ContextIntroHandler where I changed 5 to 55 in:
> for i in xrange(5):
>
>
>
> On Mon, Jan 14, 2013 at 11:33 AM, Robert Kluin <notifications@github.com>wrote:
>
>> In example/__init__.py:
>>
>> >
>> >          # Create a new furious Context.
>> >          with context.new() as ctx:
>> >              # "Manually" instantiate and add an Async object to the Context.
>> > -            async_task = Async(
>> > -                target=example_function, kwargs={'first': 'async'})
>> > -            ctx.add(async_task)
>> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>> > +            ctx.add(target=whatup, args=[randint(1,20)])#,callbacks={'success': oooeee})
>>
>> Was this intended to be committed? If it was, I would vote that we at
>> least make this 1) conform to PEP8, including line length, 2) make this
>> code a little nicer:
>>
>> for _ in xrange(20):
>>     sync_task = Async(target=whatup, args=[randint(1,20)])
>>     ctx.add(async_task)
>>
>> For callbacks, I think you could use the defaults decorator to clean this
>> up too.
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/WebFilings/furious/pull/17/files#r2637685>.
>>
>>
>
>