chore(ingest/presto-on-hive) Set enable_properties_merge to True by default #10469

dushayntAW · 2024-05-09T12:46:58Z

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

…efault

…perties to check

anshbansal · 2024-05-15T13:27:00Z

doc build failure is unrelated and will be fixed in #10507

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

vercel bot deployed to Preview May 9, 2024 13:01 View deployment

eboneil approved these changes May 10, 2024

View reviewed changes

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

dushayntAW added 2 commits May 15, 2024 11:46

chore(ingest/presto-on-hive) Set enable_properties_merge to True by d…

c58e404

…efault

chore(ingest/presto-on-hive) updated the method to ignore certain pro…

e35003e

…perties to check

dushayntAW force-pushed the fix/enable_properties_merge-default-true branch from f8b73a8 to e35003e Compare May 15, 2024 09:46

dushayntAW requested a review from eboneil May 15, 2024 09:48

vercel bot had a problem deploying to Preview May 15, 2024 09:55 Failure

anshbansal merged commit a164b70 into datahub-project:master May 15, 2024
56 of 58 checks passed