version 1.1.2.26, 2014/04/23 00:49:58
|
version 1.1.2.30, 2014/05/04 23:05:10
|
|
|
1. Binary operation from OOP. Need to add counter to binary | 1. Binary operation from OOP. Need to add counter to binary |
protocol to be able to count objects in response. Generates | protocol to be able to count objects in response. Generates |
warnings in things like messageserializer and does not work with | warnings in things like messageserializer and does not work with |
OOP right now. Corrected by converting to XML. |
OOP right now. Fixed by converting to XML. |
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. | return from the exec query function to the aggregator yet. |
3. Code for Pull part of OpenQueryInstancesRequest a) should be part of | 3. Code for Pull part of OpenQueryInstancesRequest a) should be part of |
|
|
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, diagnostics, etc. still in the code |
3. Minor TODOs, diagnostics, etc. still in the code |
4. External runtime variables. Proposing that they be fixed for this release | 4. External runtime variables. Proposing that they be fixed for this release |
rather than set by configuration. This should be discussed. Am making | rather than set by configuration. This should be discussed. Am making |
this a separate bug. See bug 9819 for the changes to cover this. | this a separate bug. See bug 9819 for the changes to cover this. |
5. Consider moving some more of the code in dispatcher from templates to |
5. Move processPull code in dispatcher from template to |
common functions which would mean adding intermediate classes in |
common function which would mean adding intermediate classes in |
CIMMessage but would reduce code size. Currently the change that |
CIMMessage but would reduce code size. This createsCIMmessage |
we should make for this is to create a new message CIMPullRequestData... |
CIMPullRequestData... between CIMRequestDataMessage and the Pull requests |
between CIMRequestDataMessage and the Pull requests since they all |
since they all have exactly the same input and output pararameters. |
have exactly the same input and output pararameters (The |
|
CIMResponseData hides the response object differences) |
|
6. Extension to avoid double move of objects in CIMResponseData (one | 6. 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). This would mean | of queue when converting (avoids the second move). This would mean |
extending the output writers to allow the count field to be supplied | extending the output writers to allow the count field to be supplied |
so they would only create output for up to the count supplied. |
so they would only create output for up to the count supplied.(Make |
9. Add more static tests (currently only OpenEnumerateInstances is covered). |
this future beyond bug 9676) |
|
9. Add more static tests (currently only OpenEnumerateInstances and |
|
OpenEnumerateInstanceNames covered). |
Low priority because most of this except exact XML covered in other | Low priority because most of this except exact XML covered in other |
tests. |
tests. note that we can only add very limited tests since a) no |
|
way to test multiple opertions in sequence(i.e. open/pull) and |
|
cannot be sure how many responses will come back in all cases even |
|
for open. |
10. Could clean up the CIMOperationRequestDecoder.cpp rejectCIMException | 10. Could clean up the CIMOperationRequestDecoder.cpp rejectCIMException |
further since there is some overlap. That is low priority. |
further since there is some overlap. Low priority. |
12.Add exception for to many entries in EnumerationContextTable. The |
11.Correct issue between operations and HTTP where we are sending |
test exists but is ignored right now. |
|
12. Correct issue between operations and HTTP where we are sending |
|
trailers with exceptions. Also correct issue where we are | trailers with exceptions. Also correct issue where we are |
sending endOfSequence and EnumerationContext in response when | sending endOfSequence and EnumerationContext in response when |
we send CIMException. |
we send CIMException. DONE Except that we really need to get |
|
rid of the trailers completely for pull operations. Need to |
|
have flag to send as non trailer (probably non_chunked flag) |
|
12.Make enumContextTable hashtable dynamic and create entirely within |
|
CIMOperationRequestDispatcher.cpp. Today it has a static size |
| |
NEXT TASKS: | NEXT TASKS: |
b. Finish and test the OpenQueryInstances | b. Finish and test the OpenQueryInstances |
|
|
timeout thread) and a place where not mutuxed, primarily the | timeout thread) and a place where not mutuxed, primarily the |
timeout thread. | timeout thread. |
| |
|
4 May 2014 |
|
1. Clean interface with EnumerationContext timeout thread |
|
2. Remove more diagnostics |
|
3. Correct issue with enumerationContext Release that was causing |
|
failures during nightly tests. |
|
|
|
2 May 2014 |
|
1. Remove Diagnostics from ResponseData.cpp |
|
2. Correct setup and start of EnumerationContextTimerThread |
|
3. Change limit on max number of enum contexts to be set from Dispatcher. |
|
4. Increase size of hashtable for enum contexts. |
|
5. Corrected issue causing failures in removeContext. Now runs through nightly |
|
tests most of the time. |
|
1 May 2014 |
|
1. Cleanup of code and variable names. |
|
2. Removed namespace variable from EnumerationContext. It was never used. |
|
3. Added test for max number of open enumerations. Right now it is fixed |
|
and set about 100. Note that this is tied to the size of hash table |
|
so that there is a issue with more dynamic setting of max and the |
|
hash table size. |
|
4. Removed several unused functions. |
|
5. Found one possible issue causing thread conflict but not the core one |
|
yet. |
|
|
|
30 April |
|
1. Added exception response to dispatcher for Limits exceeded on |
|
CreateEnumerationContext. Returns Exceeded server limits CIMException. |
|
2. Fixed possible deadlock where we were not unlocking EnumerationContext |
|
under error condition |
|
|
|
|
|
27 April - mergeout to head of tree |
|
1. Mergeout of the code to the head of tree. Documented that mergeout in |
|
the bug |
|
|
|
27 April 2014 |
|
1. Clean up in a number of areas, mostly just code without changing |
|
functionality. |
|
2. Modified CIMMessage.h slightly to clarify names of the pull intermediate |
|
messages (the common open and pull response); |
|
3. Removed a number of diagnostics from CIMResponseData |
|
4. Added new function to ResponseStressTestProvider to be able to terminate |
|
output after a defined number of objects with a CIMException. |
|
5. Removed a number of other TODOs |
|
6. Corrected a couple of errors in the server message bundle. |
|
7. Modified ResponseEncoder to only output the error for pull errors. |
|
It was errounously outputting the EndOfSequence, EnumerationContext and |
|
the error. This way, only the error is sent which is spec. |
|
8. Removed number of TODOs and cleaned up a couple of others. |
|
9. Added static tests for OpenEnumerateInstanceNames. |
|
10. Modified a number of the temporary diagnostics to be permanent by |
|
changing them to PEGASUS_DEBUG_ASSERTs |
|
|
22 April 2014 | 22 April 2014 |
1. Clean up some messages. | 1. Clean up some messages. |
2. Still trying to find issue that randomly keeps messages in timeout | 2. Still trying to find issue that randomly keeps messages in timeout |
|
|
timeout) and never do pulls after open. | timeout) and never do pulls after open. |
| |
21 April 2014 | 21 April 2014 |
1. Reorganizzed code in CIMOperationRequestDecoder.cpp ...Iparam classes. |
1. Reorganized code in CIMOperationRequestDecoder.cpp ...Iparam classes. |
Cleaned up the rejectCIMException code. | Cleaned up the rejectCIMException code. |
2. Cleanup some issues in the EnumerationContextTimeout Thread. | 2. Cleanup some issues in the EnumerationContextTimeout Thread. |
3. Fixed issue in pullop where it was using zero as default for default | 3. Fixed issue in pullop where it was using zero as default for default |