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.

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

Draft: Apache http+ranger bom #31

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

koushikr
Copy link
Collaborator

@koushikr koushikr commented Dec 29, 2023

  • Initial set of changes for movement from OkHttpClient to Apache http client, involves creating a single instance of ApacheHttpClient in the hubClient and in the simpleClient respectively, with lifecycle managed with the hub's start and stop.
  • The above helps keep the interfaces simple.
  • Added an HttpClientUtils to support client creation and also an execute method, helps provide a common interface for execute and also doesn't hold onto deserialization cost when the response object is still open, gets bytes, closes response object and then does de-ser. (The hub's getNodeList and getServiceList is typically hundreds in size)
  • Initial set of changes for ranger-bom, for modules using ranger to use a consistent version
  • Respective pom.xml changes across modules
  • Changed the respective tests, added changelog and have done version upgrades across poms for the new release.

…tp client

b) Initial set of changes for ranger-bom, for modules using ranger to use a consistent version
c) Respective pom.xml changes for (b) across modules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant