Skip to content

Conversation

@andrewazores
Copy link
Member

@andrewazores andrewazores commented Dec 10, 2025

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main


Related to #1130

Description of the change:

  • Refactoring and cleanup.
  • Add a dependency on AWS SDK netty-nio-client and quarkiverse quarkus-amazon-s3-transfer-manager to enable use of S3AsyncClient and S3TransferManager. This low-level async client and high-level client (built on the async client) enable more efficient handling of resource uploads to storage by allowing efficient parallel multipart uploads of InputStreams and smart client-side determination for file uploads whether a single PutObject upload or parallel multipart upload should be performed.

@andrewazores
Copy link
Member Author

/build_test

@github-actions
Copy link

Workflow started at 12/10/2025, 2:05:19 PM. View Actions Run.

@github-actions
Copy link

No GraphQL schema changes detected.

@github-actions
Copy link

No OpenAPI schema changes detected.

@github-actions
Copy link

CI build and push: All tests pass ✅
https://github.com/cryostatio/cryostat/actions/runs/20110164749

@andrewazores andrewazores changed the title perf(s3): refactor config properties for buffer size and part limit, use smaller and direct buffer perf(s3): refactor config properties for buffer size and part limit, avoid buffer copies Dec 10, 2025
@andrewazores
Copy link
Member Author

/build_test

@github-actions
Copy link

Workflow started at 12/10/2025, 3:57:32 PM. View Actions Run.

@github-actions
Copy link

CI build and push: At least one test failed ❌
https://github.com/cryostatio/cryostat/actions/runs/20113045657

@andrewazores
Copy link
Member Author

/build_test

@github-actions
Copy link

Workflow started at 12/10/2025, 4:01:50 PM. View Actions Run.

@github-actions
Copy link

CI build and push: At least one test failed ❌
https://github.com/cryostatio/cryostat/actions/runs/20113151637

@andrewazores
Copy link
Member Author

/build_test

@github-actions
Copy link

Workflow started at 12/11/2025, 12:18:32 AM. View Actions Run.

@github-actions
Copy link

No GraphQL schema changes detected.

@github-actions
Copy link

No OpenAPI schema changes detected.

@github-actions
Copy link

CI build and push: All tests pass ✅
https://github.com/cryostatio/cryostat/actions/runs/20122834401

@andrewazores andrewazores changed the title perf(s3): refactor config properties for buffer size and part limit, avoid buffer copies perf(s3): use higher-level AsyncClient and TransferManager for efficiency Dec 11, 2025
@andrewazores
Copy link
Member Author

/build_test

@github-actions
Copy link

Workflow started at 12/11/2025, 11:05:05 AM. View Actions Run.

@andrewazores andrewazores marked this pull request as ready for review December 11, 2025 16:05
@andrewazores andrewazores requested a review from a team as a code owner December 11, 2025 16:05
@andrewazores andrewazores requested a review from a team December 11, 2025 16:05
@github-actions
Copy link

CI build and push: All tests pass ✅
https://github.com/cryostatio/cryostat/actions/runs/20139412972

@github-actions
Copy link

No GraphQL schema changes detected.

@github-actions
Copy link

No OpenAPI schema changes detected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant