Skip to content

Commit 5f5e395

Browse files
Fix platform instance support on Druid ingestion (#12716)
Co-authored-by: rasnar <[email protected]> Co-authored-by: Sergio Gómez Villamor <[email protected]>
1 parent f14c42d commit 5f5e395

File tree

4 files changed

+81
-77
lines changed

4 files changed

+81
-77
lines changed

docs/how/updating-datahub.md

+2
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ This file documents any backwards-incompatible changes in DataHub and assists pe
2626

2727
- #12671: The `priority` field of the Incident entity is changed from an integer to an enum. This field was previously completely unused in UI and API, so this change should not affect existing deployments.
2828

29+
- #12716: Fix the `platform_instance` being added twice to the URN. If you want to have the previous behavior back, you need to add your platform_instance twice (i.e. `plat.plat`).
30+
2931

3032
### Known Issues
3133

metadata-ingestion/src/datahub/ingestion/source/sql/druid.py

+1-5
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,7 @@ def get_sql_alchemy_url(self):
5050
"""
5151

5252
def get_identifier(self, schema: str, table: str) -> str:
53-
return (
54-
f"{self.platform_instance}.{table}"
55-
if self.platform_instance
56-
else f"{table}"
57-
)
53+
return f"{table}"
5854

5955

6056
@platform_name("Druid")

0 commit comments

Comments
 (0)