Skip to content

Conversation

@swar00pduthks
Copy link

Problem

👋 Thanks for opening a pull request! Please include a brief summary of the problem your change is trying to solve, or bug fix. If your change fixes a bug or you'd like to provide context on why you're making the change, please link the issue as follows:

Closes: Security Vulnerabilities Fixes #3040

Solution

upgraded drop wizard to 4.0.13 and to JakartaEE

One-line summary:

Checklist

  • [ X] You've signed-off your work
  • [ X] Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • You've included a one-line summary of your change for the CHANGELOG.md (Depending on the change, this may not be necessary).
  • You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
  • You've included a header in any source code files (if relevant)

@boring-cyborg boring-cyborg bot added the api API layer changes label Apr 28, 2025
@boring-cyborg
Copy link

boring-cyborg bot commented Apr 28, 2025

Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md).

@netlify
Copy link

netlify bot commented Apr 28, 2025

Deploy Preview for peppy-sprite-186812 canceled.

Name Link
🔨 Latest commit 6a26a6a
🔍 Latest deploy log https://app.netlify.com/projects/peppy-sprite-186812/deploys/682a1f96b016df0008762449

@codecov
Copy link

codecov bot commented Apr 28, 2025

Codecov Report

Attention: Patch coverage is 71.27660% with 27 lines in your changes missing coverage. Please review.

Project coverage is 81.21%. Comparing base (a89b89c) to head (6a26a6a).

Files with missing lines Patch % Lines
...main/java/marquez/client/models/MarquezClient.java 0.00% 24 Missing ⚠️
api/src/main/java/marquez/MarquezApp.java 80.00% 1 Missing ⚠️
...c/main/java/marquez/api/ColumnLineageResource.java 97.36% 0 Missing and 1 partial ⚠️
...rc/main/java/marquez/tracing/TracingSQLLogger.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3056      +/-   ##
============================================
+ Coverage     81.18%   81.21%   +0.02%     
- Complexity     1506     1513       +7     
============================================
  Files           268      270       +2     
  Lines          7356     7426      +70     
  Branches        325      326       +1     
============================================
+ Hits           5972     6031      +59     
- Misses         1226     1237      +11     
  Partials        158      158              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@swar00pduthks swar00pduthks force-pushed the upgrade/dropwizard-4.0.13 branch 3 times, most recently from ccb33ae to c5a223a Compare May 11, 2025 14:22
@swar00pduthks swar00pduthks force-pushed the upgrade/dropwizard-4.0.13 branch from 8bd131b to 1c2e906 Compare May 18, 2025 17:19
merobi-hub and others added 22 commits May 18, 2025 18:24
* Arcade demo in about page of docs.

Signed-off-by: merobi-hub <[email protected]>

* Move demo to homepage component.

Signed-off-by: merobi-hub <[email protected]>

---------

Signed-off-by: merobi-hub <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
…19.1 (MarquezProject#3005)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: swar00pduthks <[email protected]>
…arquezProject#3004)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: Willy Lulciuc <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: Willy Lulciuc <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
…19.1 (MarquezProject#3005)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: swar00pduthks <[email protected]>
…arquezProject#3004)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: Willy Lulciuc <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
Signed-off-by: Willy Lulciuc <[email protected]>
Signed-off-by: swar00pduthks <[email protected]>
@swar00pduthks swar00pduthks force-pushed the upgrade/dropwizard-4.0.13 branch from 1c2e906 to 02b7771 Compare May 18, 2025 17:24
@swar00pduthks
Copy link
Author

swar00pduthks commented May 18, 2025

thijs-s added a commit to ilum-cloud/marquez that referenced this pull request Aug 30, 2025
@thijs-s
Copy link
Contributor

thijs-s commented Sep 19, 2025

@swar00pduthks Thanks for that. Awesome work!

We adopter it here https://github.com/ilum-cloud/marquez/

@swar00pduthks
Copy link
Author

@swar00pduthks Thanks for that. Awesome work!

We adopter it here https://github.com/ilum-cloud/marquez/

Not sure how long will it take to merge to main branch but good to know it's been adapted. Thanks

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

Labels

api API layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Security Vulnerabilities Fixes

4 participants