fix(ingest): cache sql is_profiling_enabled method #11665

hsheth2 · 2024-10-18T00:45:30Z

Fixes a bug with operation_config where the profiling enabled value could change partway through ingestion.

Was added in a broken state here #8489

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

github-actions · 2024-10-18T00:45:47Z

Hello @hsheth2 😄

Thank you so much for opening a pull request!

You can check out your contributor card and see all your past stats here!

hsheth2 added 2 commits October 17, 2024 17:38

fix(ingest): cache sql is_profiling_enabled method

dc2e069

fix self bug

a272a77

github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Oct 18, 2024

vercel bot deployed to Preview October 18, 2024 01:17 View deployment

mayurinehate approved these changes Oct 18, 2024

View reviewed changes

Merge branch 'master' into sql-operation-config-cache

90f975d

mayurinehate added the merge-pending-ci A PR that has passed review and should be merged once CI is green. label Oct 22, 2024

vercel bot deployed to Preview October 22, 2024 09:01 View deployment

hsheth2 merged commit 7ba77a9 into master Oct 22, 2024
73 of 100 checks passed

hsheth2 deleted the sql-operation-config-cache branch October 22, 2024 18:49