On Tue, Nov 24, 2015 at 02:15:59PM -0800, Andy Zhou wrote:
> From: Andy Zhou <az...@nicira.com>
> 
> Added ovsdb_transient_datum_from_json() to avoid size check for
> the diff datum that is transient in nature.
> Suppose a datum contains set, and the max number of elements is 2.
> If we are changing from set that contains [A, B], to a set contains
> [C, D], the diff datum will contains 4 elements [A, B, C, D].
> 
> Thus diff datum should not be constrained by the size limit. However
> the datum after diff is applied should not violate the size limit.
> 
> Signed-off-by: Andy Zhou <az...@nicira.com>
> 
> ---
> v1->v2:  avoid code duplication by using 'relaxed type' in implementing
>          ovsdb_transient_dataum_from_json(), as suggested in code
>          review:
> 
>          struct ovsdb_type relaxed_type = *type;
>          relaxed_type.n_min = 0;
>          relaxed_type.n_max = UINT_MAX;
>          return ovsdb_datum_from_json(datum,
>                                       &relaxed_type, json);

Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to