fix(ingest/athena): Fix DeprecationWarning: invalid escape sequence '\s' for testing (datahub-project#14267)

acrylJonny · web-flow · commit 002ac82d3992 · 2025-07-30T13:13:03.000-07:00
diff --git a/metadata-ingestion/src/datahub/ingestion/source/sql/athena_properties_extractor.py b/metadata-ingestion/src/datahub/ingestion/source/sql/athena_properties_extractor.py
@@ -99,10 +99,10 @@ class AthenaPropertiesExtractor:
     """A class to extract properties from Athena CREATE TABLE statements."""
 
     CREATE_TABLE_REGEXP = re.compile(
-        "(CREATE TABLE[\s\n]*)(.*?)(\s*\()", re.MULTILINE | re.IGNORECASE
+        r"(CREATE TABLE[\s\n]*)(.*?)(\s*\()", re.MULTILINE | re.IGNORECASE
     )
     PARTITIONED_BY_REGEXP = re.compile(
-        "(PARTITIONED BY[\s\n]*\()((?:[^()]|\([^)]*\))*?)(\))",
+        r"(PARTITIONED BY[\s\n]*\()((?:[^()]|\([^)]*\))*?)(\))",
         re.MULTILINE | re.IGNORECASE,
     )
 

Original file line number	Diff line number	Diff line change
`@@ -99,10 +99,10 @@ class AthenaPropertiesExtractor:`
`99`	`99`	`"""A class to extract properties from Athena CREATE TABLE statements."""`
`100`	`100`
`101`	`101`	`CREATE_TABLE_REGEXP = re.compile(`
`102`		`- "(CREATE TABLE[\s\n])(.?)(\s*\()", re.MULTILINE \| re.IGNORECASE`
	`102`	`+ r"(CREATE TABLE[\s\n])(.?)(\s*\()", re.MULTILINE \| re.IGNORECASE`
`103`	`103`	`)`
`104`	`104`	`PARTITIONED_BY_REGEXP = re.compile(`
`105`		`- "(PARTITIONED BY[\s\n]\()((?:[^()]\|\([^)]\))*?)(\))",`
	`105`	`+ r"(PARTITIONED BY[\s\n]\()((?:[^()]\|\([^)]\))*?)(\))",`
`106`	`106`	`re.MULTILINE \| re.IGNORECASE,`
`107`	`107`	`)`
`108`	`108`