forked from facebookincubator/velox
-
Notifications
You must be signed in to change notification settings - Fork 10
Unify connector config and session property names #11
Copy link
Copy link
Open
Labels
Description
Description
HiveConfig contains connector config properties and session properties. However, some of them have the connector name prefix like "hive." and some not. The prefix need to be removed. In Presto, the prefix would be removed from catalog session properties. Presto config properties are stored in etc/catalog/$catalog_name.properties. The file name already contains which catalog it is, and connector name is stored inside. There is no need to store the connector name for each config property. E.g.
connector.name=sqlserver
connection-url=jdbc:sqlserver://sqlserver;encrypt=false;
connection-user=sa
connection-password=SQLServerPass1
In the future the connector properties would be stored by connector, e.g. HiveConfig, IcebergConfig. They may share the same configs. The config names should not contain the connector name.
Reactions are currently unavailable