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

Update cxf to 4.0.4 #191

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update cxf to 4.0.4 #191

wants to merge 1 commit into from

Conversation

DaanGroot-KSO
Copy link

@DaanGroot-KSO DaanGroot-KSO commented May 21, 2024 •
edited
Loading

This fixes the OOM issues introduced by cxf 4.0.2: #154

@DaanGroot-KSO DaanGroot-KSO changed the title Update cfx to 4.0.4 Update cxf to 4.0.4 May 21, 2024
@dennis-yemelyanov
Copy link
Contributor

Is the issue still happening with the latest BingAds Java SDK (13.0.20.2 at the moment)?

@DaanGroot-KSO
Copy link
Author

@dennis-yemelyanov we have tried 13.0.20.1 which still has the memory leak, so I assume it's the same for 13.0.20.2

@dennis-yemelyanov
Copy link
Contributor

Thank you for confirming, @daangroot2. 4.0.4 has some other regressions and may cause intermittent timeouts.

To workaround the memory leak issue, could you exclude the CXF dependency and include some of the following?

  • jersey-client (for calling CampaignManagement, Bulk and Reporting services), please also see Customizing HTTP client for how to enabled things like compression, logging etc.
  • jaxws-rt (for calling CustomerManagement, Billing or AdInsight services)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DaanGroot-KSO @dennis-yemelyanov