Refactor deletion

[jphui]

Summary

Get rid of deleteWithReturn() and make returning something a part of the normal delete() call. This will eventually be propagated up to metadata-graph-assets so that a default-parameter-version of delete() can be used for simplicity.

Testing Done

adjusted unit testing

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Docs related to the changes have been added/updated (if applicable)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 66.09%. Comparing base (2de35c0) to head (0cab62a).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
...n/java/com/linkedin/metadata/dao/BaseLocalDAO.java	50.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #499      +/-   ##
============================================
+ Coverage     66.05%   66.09%   +0.04%     
- Complexity     1536     1537       +1     
============================================
  Files           137      137              
  Lines          5927     5923       -4     
  Branches        656      656              
============================================
  Hits           3915     3915              
+ Misses         1733     1730       -3     
+ Partials        279      278       -1     

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