- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.8k
WIP - memory optimization #12394
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
base: master
Are you sure you want to change the base?
WIP - memory optimization #12394
Conversation
| Skipping CI for Draft Pull Request. | 
| [APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. 
Needs approval from an approver in each of these files:
 Approvers can indicate their approval by writing  | 
8b394bf    to
    1e38aff      
    Compare
  
    Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
…est reliability Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
…in client_manager Signed-off-by: Helber Belmiro <[email protected]>
…client_manager Signed-off-by: Helber Belmiro <[email protected]>
…_manager Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
d45cbc1    to
    829bd3b      
    Compare
  
    | /retest | 
…nt`; replaced deprecated `ReadArtifact` functions with new implementation; updated dependency injection and tests Signed-off-by: Helber Belmiro <[email protected]>
6813ed1    to
    ac9b7b4      
    Compare
  
    …plementation Signed-off-by: Helber Belmiro <[email protected]>
…ient` for clarity and consistency Signed-off-by: Helber Belmiro <[email protected]>
…ace`; refactored dependency injection and updated function signatures Signed-off-by: Helber Belmiro <[email protected]>
…Interface`; updated related function signatures, workflows, and tests Signed-off-by: Helber Belmiro <[email protected]>
…PipelineClient`; updated dependency injection, function signatures, and tests Signed-off-by: Helber Belmiro <[email protected]>
…dated dependency injection, function signatures, and tests in affected components Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
| /retest | 
…nt`; updated dependency injection, method signatures, and tests across affected components Signed-off-by: Helber Belmiro <[email protected]>
…rface; updated struct naming, method signatures, and dependencies in `PipelineClientFake` Signed-off-by: Helber Belmiro <[email protected]>
…lowSaver`, `MetricsReporter`, and related components; consolidated `artifact.Client` usage under `PipelineClient` and updated tests Signed-off-by: Helber Belmiro <[email protected]>
…fresh; removed redundant `NewTokenRefresher` comment Signed-off-by: Helber Belmiro <[email protected]>
…r` and `MetricsReporter` tests; updated constructor calls accordingly Signed-off-by: Helber Belmiro <[email protected]>
…initialization for simplicity and clarity Signed-off-by: Helber Belmiro <[email protected]>
Description of your changes:
This is a work in progress.
Checklist: