lzc_receive: allow the caller to read the begin record
lzc_receive() requires that its
snapname argument is a snapshot name (contains '
'). allows to specify just a dataset name and would try to deduce the snapshot name from the stream.
I propose to allow
lzc_receive() to do the same.
That seems to be quite easy to implement, it requires only a small amount of logic, it does not require any additional system calls or any additional data from the stream.
The benefit is that the new behavior would allow to keep the snapshot names the same between the sender and receiver at zero cost, without a need to pass the names out of band.
Updated by Andriy Gapon over 8 years ago
The approach taken in the previous review request has been abandoned in favour of adding a new lzc_receive flavour that would allow the caller to read the begin record and to derive the useful information, e.g. the snapshot name, from it. See https://reviews.csiden.org/r/256/
Updated by Electric Monk over 8 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
commit 620f322510b2d6433f7f6af60fa52380c07756ad Author: Andriy Gapon <firstname.lastname@example.org> Date: 2015-11-11T00:40:10.000Z 6051 lzc_receive: allow the caller to read the begin record Reviewed by: Matthew Ahrens <email@example.com> Reviewed by: Paul Dagnelie <firstname.lastname@example.org> Approved by: Robert Mustacchi <email@example.com>