-
Notifications
You must be signed in to change notification settings - Fork 8
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.
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
Update hostname, uri
format in configuration-schema
#516
Conversation
postgresql
, mysql
, hive
hostname
format in configuration-schema
hostname
format in configuration-schema hostname, uri
format in configuration-schema
@@ -54,7 +54,7 @@ | |||
"additionalProperties": true, | |||
"required": [ | |||
"host", | |||
"port", | |||
"database", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks out of scope - is it intended?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's kind related, There is an issue on docs for displaying only required properties in sample configuration. And hive was missing database
in required properties
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fine - so long as it's deliberate and expected. It should really be flagged in the main PR description, but you've clarified it now :)
This is great @mtuchi, thanks. Please bump versions, push tags and merge and your leisure |
Summary
Updated the
Database URL
to betype: string
and update the required propertiesRef #511
Review Checklist
dev only changes don't need a changeset.