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
Initial support for Cassandra #277
Conversation
fe1efb3
to
872ca68
Compare
6ed00cc
to
f9256c5
Compare
|
||
if driver == "" || ddl == "" || uri == "" { | ||
if driver == "" || ddl == "" || uri == "" || len(host) == 0 { |
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.
inconsistent style: len(host) == 0
vs host == ""
nm. just got to where the params are defined
if uri == "" { | ||
missing = append(missing, "uri") | ||
if len(host) == 0 { |
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 could be an &&
to save one indentation level
|
||
cmd.Flags().String("username", "", "username to use when connecting") | ||
cmd.Flags().String("password", "", "password to use when connecting") | ||
cmd.Flags().StringSlice("host", []string{}, "hostname to use when connecting") |
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.
should this be hosts
instead?
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.
I don't think so... Right now it will be:
kubectl schemahero apply --host 10.1.1.1 --host 10.1.1.2
I think hosts would be weird, IMO
Signed-off-by: Marc Campbell <marc.e.campbell@gmail.com>
f9256c5
to
b99e920
Compare
This PR adds support for Cassandra schemas.
This also creates a new CRD for
datatype
. This is commonly used in cassandra and should / could also be implemented in other database engines.Docs PR is at schemahero/schemahero-docs#40