Skip to content

Conversation

@kuznetsss
Copy link
Collaborator

Fixes #1616.
Cherry pick of #1617 into develop.

For XRPLF#1616. Later should be ported to develop as well.
@kuznetsss kuznetsss requested a review from godexsoft September 6, 2024 12:31
Copy link
Collaborator

@godexsoft godexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@codecov
Copy link

codecov bot commented Sep 6, 2024

Codecov Report

Attention: Patch coverage is 20.00000% with 12 lines in your changes missing coverage. Please review.

Project coverage is 69.87%. Comparing base (44c07e7) to head (f9f66d3).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/etl/impl/SubscriptionSource.cpp 15.38% 1 Missing and 10 partials ⚠️
src/etl/LoadBalancer.cpp 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1632      +/-   ##
===========================================
- Coverage    69.92%   69.87%   -0.05%     
===========================================
  Files          260      260              
  Lines         9951     9958       +7     
  Branches      5481     5488       +7     
===========================================
  Hits          6958     6958              
+ Misses        1587     1586       -1     
- Partials      1406     1414       +8     

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

@kuznetsss kuznetsss merged commit 1d33b8e into XRPLF:develop Sep 6, 2024
@kuznetsss kuznetsss deleted the 1616_port_to_develop branch September 6, 2024 12:48
kuznetsss added a commit that referenced this pull request Sep 25, 2024
@kuznetsss kuznetsss restored the 1616_port_to_develop branch September 4, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix logging of SubscriptionSource

2 participants