search

apache / accumulo-testing

Apache Accumulo Testing
https://accumulo.apache.org
Apache License 2.0
15 stars 40 forks source link

Replace map reduce prop in continuous ingest verify #277

Closed DomGarguilo closed 4 months ago

DomGarguilo commented 7 months ago

Fixes #274

ctubbsii commented 7 months ago

When was this property changed in Hadoop? We need to be careful we're not breaking things with the earliest Hadoop version we still support.

DomGarguilo commented 7 months ago

When was this property changed in Hadoop? We need to be careful we're not breaking things with the earliest Hadoop version we still support.

Honestly I'm not sure. I am having trouble finding what version this change happened in. I do see that the following code was from 13 years ago at which point this property was already deprecated.

https://github.com/apache/hadoop/blame/833da65d8629e0bdd79097ed26033add9fc1864f/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java#L325

ctubbsii commented 4 months ago

git tag --contains 833da65d8629e0bdd79097ed26033add9fc1864f shows that it appears in at least Hadoop 3.0.0 and newer, and we don't support anything older than that anymore, so this should be good.