Feature #6051
closed
lzc_receive: allow the caller to read the begin record
100%
Description
Currently 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.
@zfs receive
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 almost 6 years ago
Oops, a symbol was "eaten" by the markup.
The first line should read:
Currently lzc_receive() requires that its snapname argument is a snapshot name (contains '@').
Updated by Andriy Gapon almost 6 years ago
Updated by Andriy Gapon over 5 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 Matthew Ahrens over 5 years ago
- Subject changed from allow lzc_receive to accept a dataset name as 'snapshot' parameter to lzc_receive: allow the caller to read the begin record
Updated by Electric Monk over 5 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
git commit 620f322510b2d6433f7f6af60fa52380c07756ad
commit 620f322510b2d6433f7f6af60fa52380c07756ad
Author: Andriy Gapon <avg@icyb.net.ua>
Date: 2015-11-11T00:40:10.000Z
6051 lzc_receive: allow the caller to read the begin record
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: Paul Dagnelie <pcd@delphix.com>
Approved by: Robert Mustacchi <rm@joyent.com>