Issue Details (XML | Word | Printable)

Key: DIRSERVER-740
Type: Task Task
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Alex Karasulu
Reporter: Alex Karasulu
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Directory ApacheDS

Re-enable thread pool max size parameter using new MINA executor

Created: 13/Sep/06 12:49 AM   Updated: 13/Sep/06 03:24 AM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0

Time Tracking:
Not Specified

Resolution Date: 13/Sep/06 03:24 AM


 Description  « Hide
Looks like there's no more ThreadPoolFilter in MINA and an Executor is used instead. I need to figure out from Trustin just how to set the max thread pool size now. I had to comment out some code in the ServerContextFactory on line 111 which needs to be re-enabled to allow for thread pool size configuration settings to hold.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Alex Karasulu made changes - 13/Sep/06 12:49 AM
Field Original Value New Value
Assignee Alex Karasulu [ akarasulu ]
Alex Karasulu made changes - 13/Sep/06 12:49 AM
Status Open [ 1 ] In Progress [ 3 ]
Alex Karasulu made changes - 13/Sep/06 12:49 AM
Fix Version/s 1.0 [ 12312043 ]
Trustin Lee added a comment - 13/Sep/06 01:14 AM
Here's the example code:

ThreadPoolExecutor executor = new ThreadPoolExecutor();
// configure the executor here
...

SocketAcceptorConfig cfg = new SocketAcceptorConfig();
...

ExecutorThreadModel threadModel = ExecutorThreadModel.getInstance("LDAP");
threadModel.setExecutor( executor );
cfg.setThreadModel( threadModel );
...

acceptor.bind(new InetSocketAddress(port), handler, cfg);

You don't need to add any ExecutorFilter by yourself.

Alex Karasulu added a comment - 13/Sep/06 03:24 AM
Fixed on committed revision 442813:

    http://svn.apache.org/viewvc?view=rev&revision=442813

Alex Karasulu made changes - 13/Sep/06 03:24 AM
Status In Progress [ 3 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Repository Revision Date User Message
ASF #442813 Wed Sep 13 03:26:39 UTC 2006 akarasulu taking advice posted by trustin for DIRSERVER-740
Files Changed
MODIFY /directory/branches/apacheds/1.0/protocol-changepw/src/main/java/org/apache/directory/server/changepw/ChangePasswordServer.java
MODIFY /directory/branches/apacheds/1.0/protocol-ntp/src/main/java/org/apache/directory/server/ntp/NtpServer.java
MODIFY /directory/branches/apacheds/1.0/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/KerberosServer.java
MODIFY /directory/branches/apacheds/1.0/server-jndi/src/main/java/org/apache/directory/server/jndi/ServerContextFactory.java

Repository Revision Date User Message
ASF #442829 Wed Sep 13 04:15:20 UTC 2006 akarasulu taking advice posted by trustin for DIRSERVER-740
Files Changed
MODIFY /directory/trunks/apacheds/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/KerberosServer.java
MODIFY /directory/trunks/apacheds/server-jndi/src/main/java/org/apache/directory/server/jndi/ServerContextFactory.java
MODIFY /directory/trunks/apacheds/protocol-ntp/src/main/java/org/apache/directory/server/ntp/NtpServer.java
MODIFY /directory/trunks/apacheds/protocol-changepw/src/main/java/org/apache/directory/server/changepw/ChangePasswordServer.java