(file) Return to readme.pulloperations CVS log (file) (dir) Up to [Pegasus] / pegasus

Diff for /pegasus/readme.pulloperations between version 1.1.2.5 and 1.1.2.9

version 1.1.2.5, 2013/06/11 23:17:40 version 1.1.2.9, 2013/09/17 16:24:34
Line 9 
Line 9 
  
 NOTES On working with task branch. NOTES On working with task branch.
  
 Merge out  Merge out Process
 I just start with new directory and put TaskMakefile above pegasus  
  
 make -f TaskMakefile BNAME=PEP317-pullop  ## takes a long time     To keep our TASK branch in sync with the current head of tree we need
      to do a regular merge out.  the TaskMakefile contains the makefile
      procedures to do this efficiently.  NOTE: Following these procedures is
      important in that you are merging out new material each time you do
      the merge out.  If you were just to repeatedly merge out, you would be
      merging previously merged changes a second time causing a real mess.
   
       Start with new directory and put TaskMakefile above pegasus (needed so you
       have this file for the initial operations.
   
         make -f TaskMakefile branch_merge_out BNAME=PEP317-pullop  ## takes a long time
   
      This checks out current head, merges it into task branch and sets tags
      for the mergeout.  Note that at the end of this step this work is
      part of the TASK... branch.
   
      NOW check for conflicts, errors, etc. that resulted from the merge.
      Look for conflict flags, compare the results (I use linux merge as a
      good graphic compare tool) and build and test. When you are satisfied
      that the merge out is clean, you can commit the results to the TASK...
      branch
  
    checks out current head, merges it into task branch and sets tags     To commit the work to  this into Task branch
    NOW check for conflicts, errors, etc.  
   
    To check this into Task branch  
  
 make -f mak/TaskMakefile branch_merge_out_commit BNAME=PEP317-pullop make -f mak/TaskMakefile branch_merge_out_commit BNAME=PEP317-pullop
  
   or manually commit and finish as follows   or manually commit and finish as follows
  
 cvs commit cvs commit
   
 make -f mak/TaskMakefile  branch_merge_out_finish BNAME=PEP317-pullop make -f mak/TaskMakefile  branch_merge_out_finish BNAME=PEP317-pullop
 ## This last step is important since it cleans up temporary tags.  
   ## This last step is important since it cleans up temporary tags to prepare
      you for the next checkout
  
 COMPARE TASKBRANCH WITH HEAD COMPARE TASKBRANCH WITH HEAD
  
 In a new pegasus work space do same as for merge out.      In a new pegasus work space do same as above for merge out.
  
 make -f TaskMakefile BNAME=PEP317-pullop make -f TaskMakefile BNAME=PEP317-pullop
  
 This produces a result which is all of the head merged into the branch. This produces a result which is all of the head merged into the branch.
 A diff of this is all the new changes to the head of tree.      A diff of this is all the new changes to the head of tree that you will
       include into the merge.
  
  
 TODO list: TODO list:
Line 53 
Line 70 
    4. The changes to WQLCIMOperationRequestDispatcher and CQL... for handling    4. The changes to WQLCIMOperationRequestDispatcher and CQL... for handling
       pull not completed so we feed the responses back to the EnmerationContext       pull not completed so we feed the responses back to the EnmerationContext
       queues       queues
    3. Lots of minor TODOs, etc.     3. Lots of minor TODOs, diagnostics, etc.
    4. External runtime variables. Decide this as part of PEP. The variables    4. External runtime variables. Decide this as part of PEP. The variables
       exist in CIMOperationRequestDispatcher but not in CIMConfig.  The primary       exist in CIMOperationRequestDispatcher but not in CIMConfig.  The primary
       ones to consider are:       ones to consider are:
Line 76 
Line 93 
       b. We should protect against providers that no not every finish delivering       b. We should protect against providers that no not every finish delivering
           or take to long between deliveries.  This does not exist in Pegasus           or take to long between deliveries.  This does not exist in Pegasus
           today           today
    6. Clean up code in Dispatcher. The associators code is still real mess     6. Clean up code in Dispatcher. Want to at least reduce the code for the
       and the pull code is in a template.  The Pull code is good now but        Open Operations to a set of templates so we know that the code is the
       must be duplicated.  Look at creating new CIMMessage CIMPullResponseMessage        same for all operations.  Right now the existing operations enum, assoc,
       so that we can have common code.  Everything is the same except what        etc. are all in templates but not the open operations.  Note that the
       goes into the CIMResponseData so it is logical to have completely        pull is a single template for both pullInstances and pullInstancePaths.
       common processing        However, we might be able to reduce this to a single function by adding
         a new level to CIMMessage.h (CommonPullResponse)
    7. Extension to avoid double move of objects in CIMResponseData (one    7. Extension to avoid double move of objects in CIMResponseData (one
       into enumerationContext queue and second to new cimResponseData for       into enumerationContext queue and second to new cimResponseData for
       response.  Want to avoid second move by extending Open/Pull response       response.  Want to avoid second move by extending Open/Pull response
       messages to include count and CIMResponse data to count objects out       messages to include count and CIMResponse data to count objects out
       of queue when converting (avoids the second move).  Big issue here       of queue when converting (avoids the second move).  Big issue here
       with binary data since need to extend format to count it.       with binary data since need to extend format to count it.
    8. Still using templates, etc. in  code in the Dispatcher. This is for     8. NEXT TASKS:
       all of the open operations where there is a lot of duplicate code        a. get the pull operations rather than a template into a single
       and the pull operations that are 99% duplicate code (in a single template)        function by creating a new CIMPullResponse message in CIMMessage.h that
         contains the pull data.  Then we can use a single function to process all
         pull operations.
         b. test the enumeration timeout thread
         c. New Mergout to bring up to to date again.
         d. Fix known problem with the interop provider.
   
   17 September 2013 CVS update (Actually two different updates over 3 days)
   1. Clean up some issues in CIMMessage.h and CIMMessage.cpp
   2. Extend OpenExecQuery to WQL and CQL processors but return not complete
   3. Remove memory leak in EnumerationContext and EnumerationContextTable
      handling.
   4. Created template functions for much of the pull operations.
   5. Reversed order of queryLanguage and query (and changed names to match
      execQuery) in client and server.  Note that these are the execQuery
      WQL and CQL filters and NOT FQL filters.
   6. Some code cleanup in dispatcher
   7. Today, passes all tests in pullop but issue in alltests. For some reason
      not finding CIMObjectManager instance. Also, leaves enumeration contexts
      if client terminates since cleanup thread not operating.
   
   14 September 2013 CVS update
   Merged out up to 25 August.  Cleaned up all operations and standardized code.
   At this point the non pull operations code is in a set of templates but the
   pull is not yet.
   Fixed a significant number of problems so that it appears that the operations
   except for OpenExecQuery run stably, at least with the pullop test program.
   Note that there is a problem in that the Interop control provider is not
   returning its singleton wbemserver object for some reason.  Causes a test
   failure
   
   Fixed for 16 June CVS Update
      1. Cleaned up the enumerationContext and Table release functions and tested
         to confirm that we do not lose memory in either normal sequences or
         sequences that close early. Cleaned up pullop and added more tests
   Taged Before: PREAUG25UPDATE and after POSTAUG25UPDATE
  
 Fixed for 9 June CVS update Fixed for 9 June CVS update
    1. Cleaned up code for OpenQueryInstances.  Note that this is incomplete.    1. Cleaned up code for OpenQueryInstances.  Note that this is incomplete.


Legend:
Removed from v.1.1.2.5  
changed lines
  Added in v.1.1.2.9

No CVS admin address has been configured
Powered by
ViewCVS 0.9.2