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.

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

DeleteElements to return count #3356

Merged
merged 10 commits into from
Jan 22, 2025
Merged

Conversation

cn337131
Copy link
Member

@cn337131 cn337131 commented Jan 21, 2025

DeleteElements will now return a count to reflect the number of elements deleted. Additional debug logging added as well to allow users to check what is deleted.

Related issue

@cn337131 cn337131 added the enhancement Improvement to existing functionality/feature label Jan 21, 2025
@cn337131 cn337131 added this to the 2.4.0 milestone Jan 21, 2025
@tb06904 tb06904 modified the milestones: 2.4.0, 2.3.2 Jan 21, 2025
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 68.08%. Comparing base (07694ea) to head (4531628).
Report is 11 commits behind head on develop.

Files with missing lines Patch % Lines
...q/gaffer/operation/impl/delete/DeleteElements.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3356      +/-   ##
=============================================
+ Coverage      68.06%   68.08%   +0.02%     
  Complexity      2596     2596              
=============================================
  Files            957      957              
  Lines          30674    30684      +10     
  Branches        3402     3405       +3     
=============================================
+ Hits           20877    20892      +15     
+ Misses          8313     8310       -3     
+ Partials        1484     1482       -2     

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

@j69772 j69772 merged commit 9a0de1b into develop Jan 22, 2025
29 checks passed
@j69772 j69772 deleted the gh-3355-delete-elements-to-return-count branch January 22, 2025 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement to existing functionality/feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DeleteElements should return number of elements deleted
4 participants