[ovs-dev] [PATCH v1 1/2] sflow: restrict sampling from 1 to UINT32_MAX

Ben Pfaff blp at ovn.org
Thu Dec 22 16:49:09 UTC 2016


On Thu, Dec 15, 2016 at 08:41:29PM -0800, Benli Ye wrote:
> When sampling field is 0, no need to generate sample or
> the inner action.

Thanks for the patch.

The change to compose_sample_action() makes sense to me.

I don't think that we should change the database schema because the
benefit of the change is minimal but it carries the risk of breaking a
few users' configurations.

Can you repost with a Signed-off-by?

Thanks,

Ben.


More information about the dev mailing list