Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ISSUE-14881: PinotDB Test Connection Success #14882

Merged
merged 4 commits into from
Feb 13, 2024
Merged

ISSUE-14881: PinotDB Test Connection Success #14882

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
281279d
ISSUE-14881: PinotDB Test Connection Success
Supan90-Shah3006 Jan 25, 2024
fb0ff6b
Update ingestion/src/metadata/ingestion/source/database/pinotdb/conne…
pmbrull Feb 12, 2024
f96a650
Update ingestion/src/metadata/ingestion/source/database/pinotdb/conne…
pmbrull Feb 12, 2024
7111d80
Merge branch 'main' into issue-14881
pmbrull Feb 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions ...rc/main/resources/json/schema/entity/services/connections/database/pinotDBConnection.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,23 +34,23 @@
},
"username": {
"title": "Username",
"description": "username to connect to the PinotDB. This user should have privileges to read all the metadata in PinotDB.",
"description": "username to connect to the PinotDB. This user should have privileges to read all the metadata in PinotDB.",
"type": "string"
},
"password": {
"title": "Password",
"description": "password to connect to the PinotDB.",
"description": "password to connect to the PinotDB.",
"type": "string",
"format": "password"
},
"hostPort": {
"title": "Host and Port",
"description": "Host and port of the PinotDB service.",
"title": "Pinot Broker Host and Port",
"description": "Host and port of the PinotDB Broker service.",
"type": "string"
},
"pinotControllerHost": {
"title": "Pinot Broker Host and Port",
"description": "Pinot Broker Host and Port of the data source.",
"title": "Pinot Controller Host and Port",
"description": "Pinot Controller Host and Port of the data source.",
"type": "string"
},
"database": {
Expand Down
4 changes: 2 additions & 2 deletions openmetadata-ui/src/main/resources/ui/public/locales/en-US/Database/PinotDB.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ password to connect to the PinotDB.
$$

$$section
### Host Port $(id="hostPort")
### Pinot Broker Host Port $(id="hostPort")

This parameter specifies the host and port of the PinotDB instance. This should be specified as a string in the format `hostname:port`. For example, you might set the hostPort parameter to `localhost:8099`.

Expand All @@ -37,7 +37,7 @@ $$
$$section
### Pinot Controller Host $(id="pinotControllerHost")

Pinot Broker Host and Port of the data source.
Pinot Controller Host and Port of the data source.
$$

$$section
Expand Down
Loading