Skip to content

Unify connector config and session property names #11

Description

@yingsu00

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.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions