datawave icon indicating copy to clipboard operation
datawave copied to clipboard

TreeEqualityVisitor needs to verify parens

Open apmoriarty opened this issue 3 years ago • 0 comments

The tree equality visitor will call the following queries equal

((_Bounded = true) && (NUM > '1' && NUM < '5'))
and
(_Bounded = true) && (NUM > '1' && NUM < '5')

The first query is correct. The second query will terminate explosively. Fix the TreeEqualityVisitor so it no longer ignores reference expressions -- at least in the case of QueryPropertyMarker nodes.

apmoriarty avatar Jun 01 '22 16:06 apmoriarty