/[Apache-SVN]
ViewVC logotype

Revision 665130


Jump to revision: Previous Next
Author: marc
Date: Tue Jun 5 22:20:14 2007 UTC (17 years, 5 months ago)
Changed paths: 7
Log Message:
Modified PosixThreadFactory
        Added explicit detached getter and setter
Modified PosixThreadFactory::~PThread:
        Check for join failing and don't transition to detached_ state if it does.  Potential thread-handle leak for
        threads created joinable who aren't referenced by any external thread.  Solution for now has to be
        "DONT DO THAT", the clever approach doesn't always work.

Added ThreadFactoryTests.floodNThreads:
        Loop M times for N threads where M x N is bigger than 32K to verify that detached threads can be created
        ad infinitum.

Reviewed By: mcslee

Revert Plan: revertible

Test Plan: concurrency_test thread-factory passes


Changed paths

Path Details
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/PosixThreadFactory.cpp modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/PosixThreadFactory.h modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/Thread.h modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/ThreadManager.cpp modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/test/Tests.cpp modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/test/ThreadFactoryTests.h modified , text changed
Directoryincubator/thrift/trunk/lib/cpp/src/concurrency/test/ThreadManagerTests.h modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26