> On Dec 8, 2017, at 12:53 PM, Ben Pfaff <b...@ovn.org> wrote:
> 
> +    if (argc > 2 && argv[1][0] == '_') {
> +        unixctl_command_reply_error(conn, "cannot compact built-in 
> databases");
> +        return;
> +    }

This error condition seems a little odd to me.  I think it will only provide 
this warning if multiple databases are specified, and only complain if the 
first one is built-in.

Acked-by: Justin Pettit <jpet...@ovn.org>

--Justin


_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to