Fix open-metadata#11468: Fix salesforce scheme issue (open-metadata#1…

…1474)
madsemildahlgaard · May 8, 2023 · e20f75f · e20f75f
1 parent 94ef30c
commit e20f75f
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 17 deletions.
diff --git a/ingestion/src/metadata/ingestion/source/database/salesforce/metadata.py b/ingestion/src/metadata/ingestion/source/database/salesforce/metadata.py
@@ -51,6 +51,8 @@
 
 logger = ingestion_logger()
 
+SALESFORCE_DEFAULT_SCHEMA = "salesforce"
+
 
 class SalesforceSource(DatabaseServiceSource):
     """
@@ -109,8 +111,7 @@ def get_database_schema_names(self) -> Iterable[str]:
         """
         return schema names
         """
-        schema_name = self.service_connection.scheme.name
-        yield schema_name
+        yield SALESFORCE_DEFAULT_SCHEMA
 
     def yield_database_schema(
         self, schema_name: str

diff --git a/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md b/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md
@@ -10,7 +10,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Properties
 
 - **`type`**: Service Type. Refer to *#/definitions/salesforceType*. Default: `Salesforce`.
-- **`scheme`**: SQLAlchemy driver scheme options. Refer to *#/definitions/salesforceScheme*. Default: `salesforce`.
 - **`username`** *(string)*: Username to connect to the Salesforce. This user should have privileges to read all the metadata in Redshift.
 - **`password`** *(string)*: Password to connect to the Salesforce.
 - **`securityToken`** *(string)*: Salesforce Security Token.
@@ -23,7 +22,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Definitions
 
 - **`salesforceType`** *(string)*: Service type. Must be one of: `['Salesforce']`. Default: `Salesforce`.
-- **`salesforceScheme`** *(string)*: SQLAlchemy driver scheme options. Must be one of: `['salesforce']`. Default: `salesforce`.
 
 
 Documentation file automatically generated at 2022-07-14 10:51:34.749986.
diff --git a/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md b/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md
@@ -10,7 +10,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Properties
 
 - **`type`**: Service Type. Refer to *#/definitions/salesforceType*. Default: `Salesforce`.
-- **`scheme`**: SQLAlchemy driver scheme options. Refer to *#/definitions/salesforceScheme*. Default: `salesforce`.
 - **`username`** *(string)*: Username to connect to the Salesforce. This user should have privileges to read all the metadata in Redshift.
 - **`password`** *(string)*: Password to connect to the Salesforce.
 - **`securityToken`** *(string)*: Salesforce Security Token.
@@ -23,7 +22,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Definitions
 
 - **`salesforceType`** *(string)*: Service type. Must be one of: `['Salesforce']`. Default: `Salesforce`.
-- **`salesforceScheme`** *(string)*: SQLAlchemy driver scheme options. Must be one of: `['salesforce']`. Default: `salesforce`.
 
 
 Documentation file automatically generated at 2022-07-14 10:51:34.749986.
diff --git a/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md b/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md
@@ -10,7 +10,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Properties
 
 - **`type`**: Service Type. Refer to *#/definitions/salesforceType*. Default: `Salesforce`.
-- **`scheme`**: SQLAlchemy driver scheme options. Refer to *#/definitions/salesforceScheme*. Default: `salesforce`.
 - **`username`** *(string)*: Username to connect to the Salesforce. This user should have privileges to read all the metadata in Redshift.
 - **`password`** *(string)*: Password to connect to the Salesforce.
 - **`securityToken`** *(string)*: Salesforce Security Token.
@@ -23,7 +22,5 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Definitions
 
 - **`salesforceType`** *(string)*: Service type. Must be one of: `['Salesforce']`. Default: `Salesforce`.
-- **`salesforceScheme`** *(string)*: SQLAlchemy driver scheme options. Must be one of: `['salesforce']`. Default: `salesforce`.
-
 
 Documentation file automatically generated at 2022-07-14 10:51:34.749986.
diff --git a/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md b/...a-standard/schemas/entity/services/connections/database/salesforceConnection.md
@@ -10,7 +10,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Properties
 
 - **`type`**: Service Type. Refer to *#/definitions/salesforceType*. Default: `Salesforce`.
-- **`scheme`**: SQLAlchemy driver scheme options. Refer to *#/definitions/salesforceScheme*. Default: `salesforce`.
 - **`username`** *(string)*: Username to connect to the Salesforce. This user should have privileges to read all the metadata in Redshift.
 - **`password`** *(string)*: Password to connect to the Salesforce.
 - **`securityToken`** *(string)*: Salesforce Security Token.
@@ -23,7 +22,6 @@ slug: /main-concepts/metadata-standard/schemas/entity/services/connections/datab
 ## Definitions
 
 - **`salesforceType`** *(string)*: Service type. Must be one of: `['Salesforce']`. Default: `Salesforce`.
-- **`salesforceScheme`** *(string)*: SQLAlchemy driver scheme options. Must be one of: `['salesforce']`. Default: `salesforce`.
 
 
 Documentation file automatically generated at 2022-07-14 10:51:34.749986.
diff --git a/...main/resources/json/schema/entity/services/connections/database/salesforceConnection.json b/...main/resources/json/schema/entity/services/connections/database/salesforceConnection.json
@@ -11,12 +11,6 @@
       "type": "string",
       "enum": ["Salesforce"],
       "default": "Salesforce"
-    },
-    "salesforceScheme": {
-      "description": "SQLAlchemy driver scheme options.",
-      "type": "string",
-      "enum": ["salesforce"],
-      "default": "salesforce"
     }
   },
   "properties": {