Bug 1972 - BufferingConstraintPolicy should be decoupled from TAO so that simple server doesn't link it
Summary: BufferingConstraintPolicy should be decoupled from TAO so that simple server ...
Status: RESOLVED FIXED
Alias: None
Product: TAO
Classification: Unclassified
Component: ORB (show other bugs)
Version: 1.5.10
Hardware: All All
: P3 enhancement
Assignee: Johnny Willemsen
URL:
Depends on:
Blocks: 2181
  Show dependency tree
 
Reported: 2004-11-01 01:49 CST by Johnny Willemsen
Modified: 2007-08-06 14:18 CDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Johnny Willemsen 2004-11-01 01:49:15 CST
Footprint todo item.

**

Johnny-

        We need to do that. But IMHO it is not a low hanging fruit. We
should concentrate on the ow-hanging ones first!

Thanks
Bala

On Friday, 29 October, 2004 at 15:23:23 +0200, Johnny Willemsen wrote:
 > Hi,
 > 
 > A question, when inspecting the symbols in a linked server, I found that the
 > BufferingConstraintPolicy any operators are in the TAO/tests/Hello/server
 > executable. Just a simple sever, so I wouldn't expect that to be in (static
 > build). This is used in tao/Sync_Strategies in some classes, do we have
 > plans to split that file into multiple files? Anyone did already something
 > in that area?
 > 
 > Any ideas?
 > 
 > Johnny
 > 
 > See
 > http://www.dre.vanderbilt.edu/~isisbuilds/auto_compile_logs/FP_Metrics/metri
 > cs/TAO___tests___Hello___server_Footprint.html to see that it is there.
Comment 1 Johnny Willemsen 2004-11-12 03:32:55 CST
accept for tao-support
Comment 2 Johnny Willemsen 2005-07-29 04:46:54 CDT
With the Any refactoring the BufferingConstraintPolicy was also causing a
problem. It had a create method that accepted an Any. This was only called from
the PolicyFactory in the Messaging library for the create_policy call. Moved the
functionality of this static create method to the PolicyFactory itself, this way
we don't use Any anymore in the core TAO lib. 

For the future we still have to move BufferingConstraintPolicy to the Messaging
lib and also move the associated sync strategies. When this is done there also
seems no need anymore for the TimeBase module in the core TAO lib
Comment 3 Johnny Willemsen 2007-08-06 06:37:57 CDT
mine, have this pending on my system, need to run a few test first 
Comment 4 Johnny Willemsen 2007-08-06 06:38:14 CDT
updated vers
Comment 5 Johnny Willemsen 2007-08-06 14:18:40 CDT
Mon Aug  6 18:31:12 UTC 2007  Johnny Willemsen  <jwillemsen@remedy.nl>

        * tao/Buffering_Constraint_Policy.{h,cpp,inl}:
        * tao/Messaging/Buffering_Constraint_Policy.{h,cpp,inl}:
          Moved the Buffering Constraint Policy to the Messaging Library. It
          doesn't need in the core, and if someone wants to create the Policy
          he already needed the Messaging library

        * tao/extra_core.mpb:
          Removed Buffering_Constraint_Policy.cpp:

        * tao/Makefile.am:
          Updated

        * tao/Messaging/Messaging.pidl:
          Include TAO_Ext.pidl

        * tao/Messaging/Messaging_ORBInitializer.cpp:
          Set the buffering constraint strategies into the ORB

        * tao/Messaging/Messaging_PolicyFactory.{h,cpp}:
          Updated for the fact that the Buffering Constraint Policy is now
          in the Messaging lib

        * tao/Transport_Queueing_Strategies.{h,cpp}:
        * tao/Messaging/Messaging_Queueing_Strategies.{h,cpp}:
          Moved the strategies that used the Buffering Constraint Policy to
          the Messaging lib

        * tao/Messaging/TAO_Ext.pidl:
        * tao/TAO.pidl:
          Moved Buffering Constraint Policy to Messaging

        * tao/ORB_Core.{h,cpp,inl}:
          Updated all queueing strategy methods to return a pointer and
          added set methods for the strategies that are now in the
          messaging lib

        * tao/Stub.{h,cpp}:
          Return the queueing strategy as pointer

        * tao/tao.mpc:
          Removed Buffering_Constraint_Policy.h

        * tao/Transport.cpp:
          Updated to handle the queuing strategy as pointer