#271 ✓resolved
Michael Klishin (antares)

[PULL] New pooling needs some critics

Reported by Michael Klishin (antares) | May 10th, 2008 @ 11:31 AM

New DO Pooling module implementation is mostly done.

I may benefit from some good critics though because

I am not really experienced with threading programming.

Grab it from

git://github.com/michaelklishin/do.git

and checkout newpooling branch.

Specs have been reorganized in an ad-hoc manner to use

Autotest without extra setups.

I really would love to hear what wrong with it. Thank you.

Comments and changes to this ticket

  • Tim Dysinger

    Tim Dysinger May 10th, 2008 @ 12:08 PM

    IMO needs rebase before it can be reviewed / merged with master.

  • Michael Klishin (antares)

    Michael Klishin (antares) May 13th, 2008 @ 09:40 AM

    Tim,

    Rebase was hard and I ended up reverting spec directory structure changes I did to use Autotest.

    It is merged with Sam's DO tree now in my master branch. Hopefully now it is easier to compare. So

    git pull git://github.com/michaelklishin/do.git newpooling-tmp

  • Sam Smoot

    Sam Smoot May 20th, 2008 @ 10:24 PM

    • State changed from “new” to “resolved”

    Merged, so closing out the ticket. I plan to review/switch DO over to this tonight.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

People watching this ticket

Pages