Replace deprecated JexlArithmetic.compare()

NationalSecurityAgency / datawave

DataWave is an ingest/query framework that leverages Apache Accumulo to provide fast, secure data access.

https://code.nsa.gov/datawave

Apache License 2.0

561 stars 243 forks source link

Replace deprecated JexlArithmetic.compare() #2490

Closed SethSmucker closed 2 weeks ago

SethSmucker commented 2 months ago

The previous version used a String instead of a JexlOperator for the operator parameter. The following classes have been updated to use JexlArithmetic.compare(Object left, Object right, JexlOperator operator):

datawave.query.jexl.DatawaveArithmetic
datawave.query.jexl.DefaultArithmetic
datawave.query.jexl.HitListArithmetic

part of https://github.com/NationalSecurityAgency/datawave/issues/2443

ivakegg commented 3 weeks ago

Why am I seeing changes to all of the docker files in the submodules in this PR. Also the formatting is failing.

SethSmucker commented 3 weeks ago

Why am I seeing changes to all of the docker files in the submodules in this PR. Also the formatting is failing.

I think that they're from changes merged from integration, but I could be wrong. I've been working on the sub-modules in their own directories, so that could be it too.