> On June 26, 2014, 8:18 a.m., Gordon Sim wrote: > > No objection to this change. However, what would be the impact of change > > the field type? I'm assuming the python tools wouldn't fail, would they? If > > we can get a solid understanding of the impact of just changing the type, > > and decide that the impact is acceptable, then it would be cleaner/simpler. > > Of course if the impact is unacceptable, this patch is fine. I'd just like > > to be explicit about what the impact is. > > Ernie Allen wrote: > Cumin's database relies on the management-schema. If we change a field > name or type, Cumin would get errors during object updates.
Also the qpid-snmp package would need to be updated. The mib would need to be regenerated and the code updated. - Ernie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22975/#review46724 ----------------------------------------------------------- On June 25, 2014, 7:18 p.m., Ernie Allen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22975/ > ----------------------------------------------------------- > > (Updated June 25, 2014, 7:18 p.m.) > > > 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 > >
