Bazaar Version Control System

new hpss encoding

Reported by Martin Pool on 2007-04-04
This bug report is a duplicate of:  Bug #83935: improvements to bzr protocol. Edit Remove
2
Affects Status Importance Assigned to Milestone
Bazaar
Undecided
Unassigned

Bug Description

We should update the smart transport encoding. The following properties would be useful:

 * include a "what encoding are we using" right at the start
 * include the client and server version respectively for logging purposes
 * parameters shouldn't be limited to single-line strings; it would be useful if they could include lists or dicts
 * the protocol and library should not require the entire body content to be held in memory, or its length to be known before transmission

This can be done compatibly with 0.15 by specially recognizing lines that look like a request from that version.

>>>>> "Martin" == Martin Pool <email address hidden> writes:

    Martin> Public bug reported:
    Martin> We should update the smart transport encoding. The following
    Martin> properties
    Martin> would be useful:

<snip/>

    Martin> * the protocol and library should not require the entire body
    Martin> content to be held in memory, or its length to be known before
    Martin> transmission

The server can do with a temporary file if we do not want it to
hold the body in memory, *but* I'm against not prepending the
length before the transmission, that would complicate things for
the client and reduce reliability for no benefits.

If some streaming should be used then each chunk should be
prepended with its length if we really don't want to block for
the whole body.

Just my two cents,

    Vincent

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers