Change Optional to Ternary in dash_acl.p4

[KrisNey-MSFT]

DASH/dash-pipeline/bmv2/dash_acl.p4

Line 22 in e39249b

#define LIST_MATCH optional

Per our Community discussion today, could we change 'optional' to 'ternary' ?

Opening Issue per @marian-pritsak , @mgheorghe , and @r12f

[KrisNey-MSFT]

This work is going to take some time and it also changes the attributes of the ACL table too (ternary match will add a priority attribute, which is not in design), so it will require a larger change to get it addressed. (Why are we not using 'priority' from the beginning)? Libraries to make this work w/bmv2 will need to be updated (template updates). We may also hit some test issues as well.
Kristina closing PR, but will keep Issue open.

[KrisNey-MSFT]

The problem is the list. Need to break into a list of ternary (a conversion).

[KrisNey-MSFT]

Either we move to use a TAG and deprecate this, or we implement an actual list and use ternary as a workaround.
Same as list error.
Keep this open until we find a contributor - lower priority until we get TAG in place.