> On Jul 26, 2019, at 8:11 AM, William Tu <u9012...@gmail.com> wrote:
>
> On Thu, Jul 25, 2019 at 04:24:04PM -0700, Yi-Hung Wei wrote:
>>
>> + "type": {"key": "string", "value": "string",
>> + "min": 0, "max": "unlimited"}}},
>> + "indexes": [["timeouts"]]},
>
> Just to let you know, this is a little different than your
> original patch. I made the timeouts column to be unique using
> "indexes" above. And update the cksum.
Hi, William. As we discussed off-line, I don't think we want to do this, since
it could complicate user interactions. It requires that user be aware of
whether they introduce two timeout policies that are identical, which could be
burdensome if they're taking user input, which could start off with two
different policies that become the same later. The controller would need to
keep track of that situation. I think it's better if ovs-vswitchd keeps track
of whether two timeout policies are identical for de-duplication if we want
that optimization.
I think Yi-Hung's v2 removed this.
Thanks,
--Justin
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
