(file) Return to review.txt CVS log (file) (dir) Up to [Pegasus] / pegasus / src / Pegasus / Common
File: [Pegasus] / pegasus / src / Pegasus / Common / Attic / review.txt (download)
Revision: 1.4, Fri Dec 28 23:56:54 2001 UTC (22 years, 6 months ago) by mday
Branch: MAIN
CVS Tags: test, pre-meta-dispatcher, pep_88, pegasus25BeforeLicenseUpdate, mday-merge-start, mday-merge-pegasus/src/Pegasus/Server, mday-merge-pegasus/src/Pegasus/Common, mday-2-0-patches, local, async_dispatcher, VERSION_2_1_RELEASE_HEAD, VERSION_2_1_RELEASE_BRANCH, VERSION_2_1_RELEASE, VERSION_2_1_1_RELEASE, VERSION_2_01_01, VERSION_2_00_RC_4, VERSION_2_00_RC_3, VERSION_2_00_RC_2, VERSION_2_00_RC_1, VERSION_2_00_BRANCH, VERSION_1_10, VERSION_1_09, VERSION_1_08, VERSION_1_07, TEST, STABLE, SLPPERFINST-root, SLPPERFINST-branch, RELEASE_2_4_FC_CANDIDATE_1, RELEASE_2_4_3, RELEASE_2_4_2, RELEASE_2_4_1-BETA3, RELEASE_2_4_1-BETA2, RELEASE_2_4_1-BETA1, RELEASE_2_4_1, RELEASE_2_4_0-RC3, RELEASE_2_4_0-RC2, RELEASE_2_4_0, RELEASE_2_4-root, RELEASE_2_4-branch, RELEASE_2_3_2-testfreeze, RELEASE_2_3_2-root, RELEASE_2_3_2-releasesnapshot, RELEASE_2_3_2-branch-freeze, RELEASE_2_3_2-branch, RELEASE_2_3_1-root, RELEASE_2_3_1-branch, RELEASE_2_3_0-root, RELEASE_2_3_0-msg-freeze, RELEASE_2_3_0-branch, RELEASE_2_2_1-snapshot, RELEASE_2_2_0_0-release, RELEASE_2_2_0-root, RELEASE_2_2_0-branch, RELEASE_2_2-root, PRE_LICENSE_UPDATE_2003, POST_LICENSE_UPDATE_2003, PEP217_PRE_BRANCH, PEP217_POST_BRANCH, PEP217_BRANCH, PEP214ROOT, PEP214BRANCH, PEP214-root, PEP214-branch, PEP213_SIZE_OPTIMIZATIONS, PEP-214B-root, PEGASUS_FC_VERSION_2_2, MONITOR_CONSOLIDATION_2_5_BRANCH, LOCAL_ASSOCPROV-ROOT, LOCAL_ASSOCPROV-BRANCH, IBM_241_April1405, CQL_2_5_BRANCH, CHUNKTESTDONE_PEP140
Changes since 1.3: +21 -0 lines 
more additions for meta dispatcher & asynchronous operations

NOTES/QUESTIONS:
================

    1.	Why do we need CIMOMHandle and CIMHandle?

    2.  Lots of warnings like this:

	warning C4101: 'e' : unreferenced local variable

    3.	DYNAMIC_LIBRARIES variable differences should be taken care of
	in mak system (not in individual makefiles; see Client/Makefile).

    4.	Use Linkage.h style everywhere.

    5.	Rename Provider2 to Provider

    6.	WBEMEXEC relies on old obsolete libraries (which have been removed).

    7.	Don't use System class directly.

    8. 	Reconcile all the queues (DQueue, MT_MessageQueue, MessageQueue, etc.).

<<< Fri Dec 28 10:11:50 2001 mdd >>>
-     MT_MessageQueue is not used

    DQueue and MessageQueue are significantly different and are both needed in Pegasus. 
    
    DQueue is a series of doubly-linked list containers that range from unlocked 
    void * containers to template classes with conditional locks and capacity controls. 
    The differences between the DQueue classes and the MessageQueue class are as follows:

 -    DQueue is a doubly-linked list container heirarchy
 -    DQueue is a container that can managed many different types of objects. 
 -    Contained objects do not need to be aware that they are contained by DQueue.
 -    Contained objects do not need to inherit from a base class.  
 -    DQueue supports traversing of the list and referring to the contained 
      objects while expecting contention; i.e., requires all traversals and 
      referrals to be done only by the owner of the queue mutex or condition
-     There is no central registry of DQueue objects, and DQueue objects are not named

DQueue and its base class is used heavily throughout the thread and ipc classes, 
and in other places where lists of objects are subject to heavy access contention.
No CVS admin address has been configured
 Powered by
ViewCVS 0.9.2