fix(ingest/fivetran): Add way to not add schema to the destination/source urn.

[treff7es]

Kafka connector/Confluent cloud connector adds the connector name to the schema, which should not be part of the urn.

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

[codecov]

Codecov Report

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

Files with missing lines	Patch %	Lines
.../src/datahub/ingestion/source/fivetran/fivetran.py	0.00%	2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (2226820) and HEAD (6d7c653). Click for more details.

HEAD has 2 uploads less than BASE

Flag	BASE (2226820)	HEAD (6d7c653)
ingestion	4	2

Files with missing lines	Coverage Δ
...on/src/datahub/ingestion/source/fivetran/config.py	82.07% <100.00%> (-14.12%)	⬇️
.../src/datahub/ingestion/source/fivetran/fivetran.py	33.92% <0.00%> (-60.62%)	⬇️

... and 201 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2226820...6d7c653. Read the comment docs.

[sgomezvillamor]

include_schema might be too generic and could lead to misunderstandings. WDYT about something like include_schema_in_urn instead?

I'm also a bit confused by the description: 'Include schema in the dataset URN' followed by 'Fivetran sets it to the connector name.' Is it referring to the dataset or the connector?