-
Couldn't load subscription status.
- Fork 2.3k
Created a new lib metadata-common and separated out IndexMetadata with required dependencies from the server #19011
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: main
Are you sure you want to change the base?
Conversation
82a812c to
b73f35c
Compare
|
❌ Gradle check result for b73f35c: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
…h required dependencies from the server Signed-off-by: Manik Garg <gargmanik1317@gmail.com>
b73f35c to
5a865ad
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #19011 +/- ##
============================================
- Coverage 72.96% 72.81% -0.16%
+ Complexity 69451 69351 -100
============================================
Files 5645 5657 +12
Lines 318787 319217 +430
Branches 46125 46140 +15
============================================
- Hits 232610 232431 -179
- Misses 67358 68023 +665
+ Partials 18819 18763 -56 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This PR is stalled because it has been open for 30 days with no activity. |
|
Can we evaluate if we can limit the |
That is not feasible as there are other common utils and classes required. Sharing a small dependency graph below. Update dependency graph of classes moved for AliasMetadata and IndexMetadata in PR description
|

Created a new lib metadata-common and separated out IndexMetadata with required dependencies from the server
Description
Related Issues
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Dependencies moved for AliasMetadata:
Dependencies moved for IndexMetadata: