Skip to content

Remove non-public org.apache.accumulo.core.clientImpl.* usages#3227

Closed
SethSmucker wants to merge 16 commits into
integrationfrom
task/deps/3168
Closed

Remove non-public org.apache.accumulo.core.clientImpl.* usages#3227
SethSmucker wants to merge 16 commits into
integrationfrom
task/deps/3168

Conversation

@SethSmucker
Copy link
Copy Markdown
Collaborator

fixes #3164 - .ClientConfConverter
fixes #3165 - .ClientContext
fixes #3166 - .ClientInfo
fixes #3167 - .ClientInfoImpl
fixes #3168 - .Credentials

part of #2443

At the moment, there are still some instances where TableLocator is the bottleneck for removing ClientInfo, ClientConfConverter and ClientContext, but this is only in two places. I've marked these with "//SETH NOTE" in the draft.

@SethSmucker
Copy link
Copy Markdown
Collaborator Author

Stacked PR Notice

This PR is stacked on #3253 and shares files with it.

PR Title Status
#3253 Remove interconnected IMA, Thrift, and ClientImpl BASE - must merge first
#3345 Replace InMemoryInstance with InMemoryAccumulo Also stacked on #3253
#3227 Remove non-public clientImpl.* usages THIS PR (DRAFT)

Shared files:

  • InMemoryAccumuloClient.java
  • InMemoryClientInfo.java
  • InMemoryConnector.java
  • InMemoryInstance.java
  • BulkInputFormat.java

IMPORTANT: Do NOT merge this until #3253 is merged. After #3253 merges, update this branch:

git checkout task/deps/3168
git merge origin/integration
git push

@SethSmucker
Copy link
Copy Markdown
Collaborator Author

This PR addresses tangled issues that must be fixed together.

Parent Issue: #3366 (Tangled: clientImpl APIs)

See #3366 for explanation of why these issues are interdependent.

@SethSmucker
Copy link
Copy Markdown
Collaborator Author

Superseded by #3253 and #3449 which cover all the clientImpl cleanup this PR was addressing.

@SethSmucker SethSmucker closed this May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant