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

Diff for /pegasus/readme.pulloperations between version 1.1.2.31 and 1.1.2.34

version 1.1.2.31, 2014/05/09 16:46:15 version 1.1.2.34, 2014/05/29 23:55:34
Line 17 
Line 17 
       Leave this as FUTURE       Leave this as FUTURE
    2. OpenExecQuery - Code is incomplete in that it does not include the    2. OpenExecQuery - Code is incomplete in that it does not include the
       return from the exec query function to the aggregator yet. Code for Pull       return from the exec query function to the aggregator yet. Code for Pull
       part of OpenQueryInstancesRequest a) should be part of        part of OpenQueryInstancesRequest a) should be part of the common
       the common CIMOperationRequestDispatcher execCommon code.        CIMOperationRequestDispatcher execCommon code. DONE, not tested. 29 May
    3. The changes to WQLCIMOperationRequestDispatcher and CQL... for handling    3. 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. DONE Not tested 29 May
    4. Minor TODOs, diagnostics, etc. still in the code    4. Minor TODOs, diagnostics, etc. still in the code
    5. Extension to avoid double move of objects in CIMResponseData (one    5. 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
Line 38 
Line 38 
       way to test multiple operations in sequence(i.e. open/pull) and       way to test multiple operations in sequence(i.e. open/pull) and
       cannot be sure how many responses will come back in all cases even       cannot be sure how many responses will come back in all cases even
       for open.       for open.
    7. Could clean up the CIMOperationRequestDecoder.cpp rejectCIMException     7. Clean up the CIMOperationRequestDecoder.cpp rejectCIMException
       further since there is some overlap.  VERY Low priority.       further since there is some overlap.  VERY Low priority.
    8. Make enumContextTable hashtable dynamic and create entirely within     8. Correct issue between operations and HTTP where we are sending
       CIMOperationRequestDispatcher.cpp. Today it has a static size  
    9. Correct issue between operations and HTTP where we are sending  
       trailers with exceptions. Modify response mechanisms so that we       trailers with exceptions. Modify response mechanisms so that we
       set non-chunked for all responses where we send error responses to       set non-chunked for all responses where we send error responses to
       avoid the trailers.       avoid the trailers.
    10. It would be more efficient in dispatcher to always use exception for     9. It would be more efficient in dispatcher to always use exception for
        rejects and change the _reject functions so that they never return        rejects and change the _reject functions so that they never return
        when they reject. This is VERY LOW PRIORITY and primarily saves        when they reject. This is VERY LOW PRIORITY and primarily saves
        a few lines of code in the reject functions and their calls.  Means we        a few lines of code in the reject functions and their calls.  Means we
Line 60 
Line 58 
            return true            return true
        }        }
        It would mean that the method trace for the handlers would not return        It would mean that the method trace for the handlers would not return
        an exit if we rejected. LOW PRIORITY         an exit if we rejected. VERY LOW PRIORITY
     11. We have reduced the number of template function greatly.  However there      10. There are still a couple of template functions around the task of
        are still several around the task of distributing requests to the         distributing requests to the multiple providers.
        multiple providers. In fact there are two similiar but different for         In fact there are two similiar but different templates for
        the associators(i.e. assoc, ref, etc.) functions and the parallel        the associators(i.e. assoc, ref, etc.) functions and the parallel
        openAssoc functions.  It would be nice to consolidate that logic and        openAssoc functions.  It would be nice to consolidate that logic and
        further to try to create a non-template form for those functions. LOW        further to try to create a non-template form for those functions. LOW
        PRIORITY.        PRIORITY.
     13. External runtime variables. Proposing that they be fixed for this release      11. External runtime variables. This bug leaves these variables as
       rather than set by configuration. Making this a separate bug.        internal to CIMOperationRequestDispatcher.cpp.
       See bug 9819 for the changes to cover this.        See bug 9819 for the changes to externalize these variables, both
         the runtime and buildtime variables.  The Work group discussion
         documents what the variables are and how they tie to statistics info.
         See the OpenPegasus Wiki, Pull operations work group.
       12. Not directly related to pull but now over half the code in
         WQLOperationRequestDispatcher.cpp and CQLOperationRequestDispatcher
         handle functions is just duplicated.  Need to create a common function
         and eliminate the duplicated code
       13. Left almost duplicate createContext function in EnumerationContextTab.
         Clean up code
   
  
    NEXT TASKS:    NEXT TASKS:
       a. finish 8, 9 3 above        a. finish 8, 11 2,3 above ( Finish by about 1 June)
   
   29 May 2014
   1. Correct issue in cimcli that was causing rejects of open operations when
      there were no filters requested. The standard default is WQL in cimcli
      so we overrode this for the pull operations.
   2. Clean up OpenQueryInstances in CIMMessage.h and CIMMessage.cpp.
   3. Modify CIMOperationRequestDispatcher and
      EnumerationContextTable.CreateContext to pass request rather than request
      parameters.
   4. Remove a number of diagnostic messages.  Note that there are still some.
      They are marked KS_TODO with with note to remove before checkin. They are
      all traces and validations.
   5. Correct error in inclusion of execinfo.h include in CIMOperationRequestDis.
      found by Ji Lee.
   6. Cleaned up execQuery and extended it for OpenQueryInstances.  ExecQuery
      tested but have not tested OpenQueryInstances yet. One more checkin and we
      should be able to close 2,3 above (OpenQueryInstances).
   
   26 May 2014
   1. Remove a number of diagnostics.
   2. Make cache size in EnumerationContextTable dynamic and remove function
      to set default values. MaximumEnums set from Dispatcher on constructor for
      EnumerationContextTable and table size set to 1/2 of maximum number of
      open contexts.
   3. Clean up some statistics in EnumerationContextTable
   4. Correct error in query and queryLanguage parameters in cimcli in that
      the default for queryLanguage is WQL but must be "" for pull operations.
      If WQL found they set it to "".
   
  
 9 May 2104 9 May 2104
 1. Converted several  template functions in CIMOperationRequestDispatcher 1. Converted several  template functions in CIMOperationRequestDispatcher
Line 106 
Line 143 
    client really did send an erronous request.    client really did send an erronous request.
 7. Modified PullErrors to correctly test above changed in behavior. 7. Modified PullErrors to correctly test above changed in behavior.
 8. Removed a significant number of diagnostic messages. 8. Removed a significant number of diagnostic messages.
   9. Added query and querylanguage parameters to cimcli.
  
  
 4 May 2014 4 May 2014

Legend:
Removed from v.1.1.2.31  
changed lines
  Added in v.1.1.2.34
No CVS admin address has been configured
 Powered by
ViewCVS 0.9.2