Skip to content

Describe using a configuration class in a connector#13001

Merged
raunaqmorarka merged 2 commits intotrinodb:masterfrom
nineinchnick:dev-guide-config
Jul 27, 2022
Merged

Describe using a configuration class in a connector#13001
raunaqmorarka merged 2 commits intotrinodb:masterfrom
nineinchnick:dev-guide-config

Conversation

@nineinchnick
Copy link
Copy Markdown
Member

Description

Describe, with examples, how to define supported configuration properties for a custom connector as a Java class and how it makes a connector more robust.

Is this change a fix, improvement, new feature, refactoring, or other?
improvement

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)
docs

How would you describe this change to a non-technical end user or system administrator?
n/a

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
(x) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Jun 27, 2022
@github-actions github-actions bot added the docs label Jun 27, 2022
@nineinchnick nineinchnick added the no-release-notes This pull request does not require release notes entry label Jun 28, 2022
@nineinchnick nineinchnick force-pushed the dev-guide-config branch 2 times, most recently from c287f2a to 6f58dd5 Compare July 22, 2022 17:35
@nineinchnick nineinchnick requested review from hashhar and mosabua July 26, 2022 11:15
@raunaqmorarka raunaqmorarka merged commit de86723 into trinodb:master Jul 27, 2022
@github-actions github-actions bot added this to the 392 milestone Jul 27, 2022
@nineinchnick nineinchnick deleted the dev-guide-config branch November 2, 2022 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed docs no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

2 participants