[asterisk-dev] [Code Review] 2704: Expose res_sip threadpool options

[opticron]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2704/
-----------------------------------------------------------

(Updated Aug. 2, 2013, 1:26 p.m.)


Review request for Asterisk Developers.


Changes
-------

Address Mark's and David's comments.


Bugs: ASTERISK-22143
    https://issues.asterisk.org/jira/browse/ASTERISK-22143


Repository: Asterisk


Description
-------

Expose initial size, maximum size, and idle timeout as configurable parameters for the res_sip thread pool.


Diffs (updated)
-----

  /trunk/res/ari/ari_model_validators.c 396150 
  /trunk/res/ari/ari_model_validators.h 396150 
  /trunk/main/stasis_bridges.c 396150 
  /trunk/main/strings.c 396150 
  /trunk/main/rtp_engine.c 396150 
  /trunk/main/manager_bridges.c 396150 
  /trunk/main/pickup.c 396150 
  /trunk/main/channel_internal_api.c 396150 
  /trunk/main/json.c 396150 
  /trunk/main/channel.c 396150 
  /trunk/main/bridge_channel.c 396150 
  /trunk/main/cel.c 396150 
  /trunk/include/asterisk/strings.h 396150 
  /trunk/main/astobj2.c 396150 
  /trunk/include/asterisk/stasis_bridges.h 396150 
  /trunk/include/asterisk/pickup.h 396150 
  /trunk/include/asterisk/json.h 396150 
  /trunk/include/asterisk/parking.h 396150 
  /trunk/include/asterisk/channel.h 396150 
  /trunk/include/asterisk/astobj2.h 396150 
  /trunk/channels/sig_pri.c 396150 
  /trunk/funcs/func_channel.c 396150 
  /trunk/channels/chan_sip.c 396150 
  /trunk/channels/chan_skinny.c 396150 
  /trunk/channels/chan_oss.c 396150 
  /trunk/channels/chan_misdn.c 396150 
  /trunk/channels/chan_iax2.c 396150 
  /trunk/channels/chan_mgcp.c 396150 
  /trunk/channels/chan_dahdi.c 396150 
  /trunk/apps/app_chanspy.c 396150 
  /trunk/res/ari/ari_websockets.c 396150 
  /trunk/res/ari/resource_asterisk.h 396150 
  /trunk/res/ari/resource_asterisk.c 396150 
  /trunk/res/ari/resource_bridges.h 396150 
  /trunk/res/ari/resource_bridges.c 396150 
  /trunk/res/ari/resource_events.h 396150 
  /trunk/res/ari/resource_events.c 396150 
  /trunk/res/ari/resource_sounds.h 396150 
  /trunk/res/parking/parking_bridge.c 396150 
  /trunk/res/parking/parking_manager.c 396150 
  /trunk/res/res_ari_asterisk.c 396150 
  /trunk/res/res_ari_bridges.c 396150 
  /trunk/res/res_ari_channels.c 396150 
  /trunk/res/res_ari_endpoints.c 396150 
  /trunk/res/res_ari_events.c 396150 
  /trunk/res/res_ari_playback.c 396150 
  /trunk/res/res_ari_recordings.c 396150 
  /trunk/res/res_ari_sounds.c 396150 
  /trunk/res/res_pjsip.c 396150 
  /trunk/res/res_pjsip/config_system.c 396150 
  /trunk/res/res_pjsip/include/res_pjsip_private.h 396150 
  /trunk/res/res_pjsip/pjsip_options.c 396150 
  /trunk/res/res_pjsip_notify.c 396150 
  /trunk/res/res_pjsip_outbound_registration.c 396150 
  /trunk/res/res_sorcery_astdb.c 396150 
  /trunk/res/snmp/agent.c 396150 
  /trunk/rest-api-templates/ari_resource.h.mustache 396150 
  /trunk/rest-api-templates/param_cleanup.mustache 396150 
  /trunk/rest-api-templates/param_parsing.mustache 396150 
  /trunk/rest-api-templates/res_ari_resource.c.mustache 396150 
  /trunk/rest-api/api-docs/asterisk.json 396150 
  /trunk/rest-api/api-docs/bridges.json 396150 
  /trunk/rest-api/api-docs/channels.json 396150 
  /trunk/rest-api/api-docs/events.json 396150 
  /trunk/rest-api/api-docs/sounds.json 396150 
  /trunk/tests/test_json.c 396150 
  /trunk/utils/Makefile 396150 
  /trunk/utils/refcounter.c 396150 

Diff: https://reviewboard.asterisk.org/r/2704/diff/


Testing
-------

Made sure that Asterisk came up and calls still worked.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130802/2e1124da/attachment.htm>