You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "DomGarguilo (via GitHub)" <gi...@apache.org> on 2024/04/15 12:33:52 UTC
[PR] Replace map reduce prop in continuous ingest verify [accumulo-testing]
DomGarguilo opened a new pull request, #277:
URL: https://github.com/apache/accumulo-testing/pull/277
Fixes #274
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Replace map reduce prop in continuous ingest verify [accumulo-testing]
Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #277:
URL: https://github.com/apache/accumulo-testing/pull/277#issuecomment-2064974557
> 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
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Replace map reduce prop in continuous ingest verify [accumulo-testing]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #277:
URL: https://github.com/apache/accumulo-testing/pull/277#issuecomment-2245838281
`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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Replace map reduce prop in continuous ingest verify [accumulo-testing]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #277:
URL: https://github.com/apache/accumulo-testing/pull/277#issuecomment-2061560061
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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Replace map reduce prop in continuous ingest verify [accumulo-testing]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii merged PR #277:
URL: https://github.com/apache/accumulo-testing/pull/277
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org