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

[bugfix](thirdparty) Upgrade aws s3 sdk to prevent mem leak #25106

Merged
merged 2 commits into from
Oct 9, 2023

Conversation

ByteYue
Copy link
Contributor

@ByteYue ByteYue commented Oct 8, 2023

Proposed changes

Issue Number: close #xxx
During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@ByteYue
Copy link
Contributor Author

ByteYue commented Oct 8, 2023

run buildall

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

clang-tidy review says "All clean, LGTM! 👍"

@ByteYue ByteYue changed the title [bugfix](thirdpartu) Upgrade aws s3 sdk to prevent mem leak [bugfix](thirdparty) Upgrade aws s3 sdk to prevent mem leak Oct 8, 2023
Copy link
Member

@airborne12 airborne12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

PR approved by anyone and no changes requested.

@morningman
Copy link
Contributor

run buildall

Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Oct 9, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2023

PR approved by at least one committer and no changes requested.

@morningman morningman merged commit e1b9854 into apache:master Oct 9, 2023
ByteYue added a commit to ByteYue/doris that referenced this pull request Oct 13, 2023
…5106)

During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.
ByteYue added a commit to ByteYue/doris that referenced this pull request Oct 13, 2023
…5106)

During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.
ByteYue added a commit to ByteYue/doris that referenced this pull request Oct 23, 2023
…5106)

During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.
ByteYue added a commit to ByteYue/doris that referenced this pull request Oct 23, 2023
dutyu pushed a commit to dutyu/doris that referenced this pull request Oct 28, 2023
…5106)

During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.
@xiaokang xiaokang removed the dev/2.0.3 label Nov 1, 2023
ByteYue added a commit to ByteYue/doris that referenced this pull request Nov 12, 2023
…5106)

During the use of the AWS S3 SDK, we found that there is a memory leak. According to the official issue, upgrading the SDK should resolve the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. dev/2.0.4-merged merge_conflict reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants