Re: [PATCH] ConnStateData flexible transport support

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Tue, 29 Apr 2014 08:19:20 -0600

On 04/29/2014 06:03 AM, Amos Jeffries wrote:

> I'm fine with "transferProtocol" or just "protocol" (although that may
> get as common as "conn" has become). Whichever suits you.

Let's avoid "protocol" for the reasons you mentioned and use
"transferProtocol".

>>> This variable can
>>> be altered whenever necessary to cause an on-wire protocol change.

>> Altering the data member does not cause an on-wire protocol change in
>> the patched code AFAICT. Perhaps you meant that the data member should
>> always reflect the current wire protocol?

> Yes. However I was planning to use it to decode which Packer was
> allocated.

Sure, and the new data member is already used for other things. To the
extent possible, the description should focus on the meaning, not use,
especially when the use cases are diverse.

Thank you,

Alex.
Received on Tue Apr 29 2014 - 14:19:45 MDT

This archive was generated by hypermail 2.2.0 : Tue Apr 29 2014 - 12:00:16 MDT