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.

Sign up for GitHub

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

Jump to bottom

fix(ingest/gc): minor tweak gc source #12093

Merged

anshbansal merged 1 commit into master from ab-minor-tweak-gc-source

Dec 11, 2024

+6 −6

Collaborator

anshbansal commented Dec 11, 2024

dataprocess_cleanup can take some time. Making it the last step so other steps get done
increasing hard delete entities by default from 10k to 25k

Checklist

The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
Links to related issues (if applicable)
Tests for the changes have been added/updated (if applicable)
Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub


          fix(ingest/gc): minor tweak gc source

08ee479

github-actions bot added the ingestion label

datahub-cyborg bot added the needs-review label

treff7es approved these changes

View reviewed changes

datahub-cyborg bot added pending-submitter-merge and removed needs-review labels

vercel bot deployed to Preview

December 11, 2024 11:35

View deployment

anshbansal merged commit 54d53cf into master

95 of 128 checks passed

anshbansal deleted the ab-minor-tweak-gc-source branch

December 11, 2024 12:54

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

Labels

ingestion pending-submitter-merge