----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22975/ -----------------------------------------------------------
Review request for qpid, Alan Conway and Gordon Sim. Repository: qpid Description ------- AMQP 0-10 allows session names to be up to 2^16 bytes long; the QMF management schema for the broker however defines the name of a session object as being up to 2^8 bytes long. If a session is created with a name greater than 256 bytes, the broker cannot send out management objects for it i.e. periodic processing fails. This patch adds a new field in the management schema that handles the full length name. The existing name field is truncated if needed. Some of the management tools could be updated to use the new field. qpid-tool works without modifications. Diffs ----- /trunk/qpid/cpp/src/qpid/broker/SessionState.cpp 1605518 /trunk/qpid/cpp/src/qpid/broker/amqp/ManagedSession.cpp 1605518 /trunk/qpid/cpp/src/qpid/broker/management-schema.xml 1605518 Diff: https://reviews.apache.org/r/22975/diff/ Testing ------- - Added session name that was 1024 characters long. - Checked the message log to verify the absence of the following exception [System] error Exception thrown by timer task ManagementAgent::periodicProcessing: Could not encode string of 1061 bytes as uint8_t string. (/home/eallen/current/qpid/cpp/src/qpid/framing/Buffer.cpp:246) - Ran qpid-tool and saw the presence of the truncated session name in the existing field, and the full name in the new field. The new field is named "fullName". Thanks, Ernie Allen
