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

Merge 'file-compression' branch into main (rebased after Spring Boot upgrade) #1440

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

Merge 'file-compression' branch into main (rebased after Spring Boot upgrade) #1440

wants to merge 90 commits into from

Conversation

bennavapbc
Copy link
Collaborator

@bennavapbc bennavapbc commented Feb 14, 2025 •
edited
Loading

🎫 Ticket

Related PR

🛠 Changes

  • Add utilities for compressing job output files to gzip format
  • Allow bulk download API to return compressed or uncompressed files (per request header)
  • Merge file-compression branch into branch based on AB2D-6340/Upgrade_spring

ℹ️ Context

  • Compress job output files to reduce file size by ~90% and (slightly) improve transfer times for clients requesting compressed files

SonarQube

image

🧪 Validation

TODO -- Need to re-test

smirnovaae and others added 30 commits October 15, 2024 13:17
@smirnovaae
Spring boot upgrade
ab77804
@Sadibhatla
Merge branch 'main' into AB2D-6340/Upgrade_spring
7ca89dc
@smirnovaae
Spring boot upgrade
75c5353
@smirnovaae
resolving conflicts
39d5963
@smirnovaae
Merge remote-tracking branch 'origin/AB2D-6340/Upgrade_spring' into A…
3566290 
…B2D-6340/Upgrade_spring

# Conflicts:
#	api/src/test/java/gov/cms/ab2d/api/controller/BulkDataAccessAPIUnusualDataTests.java
#	api/src/test/java/gov/cms/ab2d/api/controller/v2/StatusAPIV2Test.java
#	pom.xml
@smirnovaae
fix build
c5e0d07
@smirnovaae
fix build
25313bb
@smirnovaae
Merge branch 'main' into AB2D-6340/Upgrade_spring
c2f948e
@smirnovaae
wip
07bca6c
@smirnovaae
wip
3961cb4
@smirnovaae
wip
2f327ce
@smirnovaae
Spring boot upgrade fix for tests
adfb1c2
@smirnovaae
Merge branch 'main' into AB2D-6340/Upgrade_spring
d91d24f
@smirnovaae
Fix build for e2e
890649f
@smirnovaae
Merge remote-tracking branch 'origin/AB2D-6340/Upgrade_spring' into A…
e1bde1e 
…B2D-6340/Upgrade_spring
@smirnovaae
Fix build for e2e
e1b491c
@smirnovaae
Fix build for e2e
c12935c
@smirnovaae
Fix build for e2e
9979cb3
@smirnovaae
wip
c65de74
@bennavapbc
Try to fix missing BFDClient bean
8cdd581
@smirnovaae
added @componentscan for e2e-bfd-test
152923c
@smirnovaae
removed mock for Ab2dEnvironment
2a88402
@smirnovaae
added @EnableRetry
ca80e9d
@smirnovaae
added @Profile for ContractService
2ee8e26
@smirnovaae
added SNS Service
c68b7d6
@smirnovaae
Updated bfdVersion to support HttpPost with Spring boot upgrade
b6c978a
@smirnovaae
Merge branch 'main' into AB2D-6340/Upgrade_spring
cf0fcf8
@smirnovaae
wip
abfea91
@bennavapbc
Refactor, move modules around -- WIP
4441649
@bennavapbc
Update gitignore to allow test gzip file to be committed, fix formatt…
82cd1b5 
…ing in test
@bennavapbc
Copy link
Collaborator Author

This is a draft PR only; still need to re-test functionality.

Base automatically changed from AB2D-6340/Upgrade_spring to main February 15, 2025 01:53
@bennavapbc bennavapbc marked this pull request as ready for review February 18, 2025 22:40
@bennavapbc bennavapbc requested a review from a team as a code owner February 18, 2025 22:40
@bennavapbc bennavapbc changed the title Merge 'file-compression' branch into branch based on 'AB2D-6340/Upgrade_spring' Merge 'file-compression' branch into main (rebased after Spring Boot upgrade) Feb 20, 2025
@bennavapbc bennavapbc mentioned this pull request Mar 4, 2025
Closed
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.
3 participants
@bennavapbc @smirnovaae @Sadibhatla