Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove uses of deprecated and non-public accumulo apis #2443

Open
ivakegg opened this issue Jun 26, 2024 · 1 comment
Open

Remove uses of deprecated and non-public accumulo apis #2443

ivakegg opened this issue Jun 26, 2024 · 1 comment
Assignees
@lbschanno @SethSmucker

Comments

@ivakegg
Copy link
Collaborator

ivakegg commented Jun 26, 2024

Se need to do a scrub of datawave and remove any uses of non-public and deprecated accumulo apis. Note that for non-public apis, there may be situations where that cannot be done and we can subsequently request that accumulo make something public for us.
This was referenced Jul 31, 2024
Open
Closed
Merged
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Replace usage of deprecated AccumuloInputFormat
165efed 
Replace usages of
org.apache.accumulo.core.client.mapreduce.AccumuloInputFormat with the
recommended org.apache.accumulo.hadoop.mapreduce.AccumuloInputFormat.

Part of #2443
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Update call to deprecated FileSystem.isFile(Path path)
02bc5ba 
Replace a call to the deprecated FileSystem.isFile(Path path) with
FileSystem.getFileStatus(Path path).

Part of #2443
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Replace deprecated GnuParser with DefaultParser
a75321d 
The class GnuParser is deprecated. Replace usages with DefaultParser
instead.

Part of #2443.
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Update deprecated usages of SequenceFile.createWriter()
d393f49 
Update several deprecated calls to SequenceFile.createWriter() to use
SequenceFile.createWriter(Configuration conf,
SequenceFile.Writer.Option... opts) instead.

Part of #2443
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Open
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Remove references to deprecated ZooKeeperInstance
f1022ce 
Remove several references to the deprecated class ZooKeeperInstance.

Part of #2443.
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Replace deprecated usages of new Job(Configuration)
6a4645d 
Update usages of new Job(Configuration) with
Job.getInstance(Configuration).

Part of #2443
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
lbschanno added a commit that referenced this issue Aug 6, 2024
@lbschanno
Use AccumuloClient instead of deprecated Connector
5f0fa4d 
Connector is now deprecated. Use AccumuloClient instead.

Part of #2443
@lbschanno lbschanno mentioned this issue Aug 6, 2024
Merged
@lbschanno
Copy link
Collaborator

lbschanno commented Aug 6, 2024 •
edited
Loading

List of PRs created for this ticket. Subject to update.

This was referenced Aug 15, 2024
Closed
Closed
Merged
Merged
Merged
This was referenced Aug 22, 2024
Merged
Closed
Open
ivakegg pushed a commit that referenced this issue Sep 4, 2024
@lbschanno
Replace deprecated GnuParser with DefaultParser (#2498)
f017ce7 
The class GnuParser is deprecated. Replace usages with DefaultParser
instead.

Part of #2443.
ivakegg pushed a commit that referenced this issue Sep 4, 2024
@lbschanno
Remove references to deprecated ZooKeeperInstance (#2500)
3dd1e2a 
Remove several references to the deprecated class ZooKeeperInstance.

Part of #2443.
ivakegg pushed a commit that referenced this issue Sep 4, 2024
@lbschanno
Replace deprecated usages of new Job(Configuration) (#2501)
80bd3f7 
Update usages of new Job(Configuration) with
Job.getInstance(Configuration).

Part of #2443
lbschanno added a commit that referenced this issue Sep 6, 2024
@lbschanno
Replace usage of ScannerOptions
b8e079d 
Update SessionOptions to fully implement ScannerBase instead of
extending ScannerOptions. This is done to avoid using ScannerOptions
since it is not from Accumulo's public API.

Part of work for #2443
@lbschanno lbschanno mentioned this issue Sep 6, 2024
Open
This was referenced Sep 10, 2024
Merged
Merged
Open
This was referenced Sep 18, 2024
Merged
Open
This was referenced Sep 27, 2024
Merged
Open
Open
Open
Open
lbschanno added a commit that referenced this issue Oct 4, 2024
@lbschanno
Replace usage of non-public Accumulo API PeekingIterator
720c51c 
org.apache.accumulo.core.util.PeekingIterator is part of Accumulo's
non-public API. Replace all usages of it with
org.apache.commons.collections4.iterators.PeekingIterator.

Part of work for #2443
@lbschanno lbschanno mentioned this issue Oct 4, 2024
Merged
@keith-turner keith-turner mentioned this issue Oct 7, 2024
Open
alerman added a commit that referenced this issue Oct 10, 2024
@lbschanno @alerman @ddanielr
Replace usage of deprecated AccumuloInputFormat (#2496)
279c8a0 
* Replace usage of deprecated AccumuloInputFormat

Replace usages of
org.apache.accumulo.core.client.mapreduce.AccumuloInputFormat with the
recommended org.apache.accumulo.hadoop.mapreduce.AccumuloInputFormat.

Part of #2443

* Format calls to builders

* Fix formatter tag typo

---------

Co-authored-by: alerman <[email protected]>
Co-authored-by: Daniel Roberts <[email protected]>
ivakegg pushed a commit that referenced this issue Nov 1, 2024
@lbschanno
Replace usage of non-public Accumulo API PeekingIterator (#2594)
dfbbb41 
org.apache.accumulo.core.util.PeekingIterator is part of Accumulo's
non-public API. Replace all usages of it with
org.apache.commons.collections4.iterators.PeekingIterator.

Part of work for #2443
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lbschanno lbschanno

@SethSmucker SethSmucker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ivakegg @lbschanno @SethSmucker