-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Allow specifying datastore driver by config, relocate upgrade detection, mock datastore #165
Conversation
if err != nil { | ||
t.Error(err) | ||
return | ||
} | ||
datastore := ds.(*pgSQL) |
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.
you should put this below the defer
b5e8be9
to
f8b3b16
Compare
f8b3b16
to
599cac4
Compare
// Generate a pagination key if none is provided. | ||
// TODO(Quentin-M): Move to the API code. |
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 doesn't make sense to move to the API as it is setting a default value for the config.
599cac4
to
3f49fa4
Compare
// If no Namespace is associated, | ||
// - If the FeatureVersion is present in the parent layer, associate its Namespace, | ||
// - If a Namespace has been detected in the layer, associate it, | ||
// - Otherwise, throw an error. |
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.
Let's rework this comment because it's really hard to understand.
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.
Let's add a comment addressing the building of this map and then we'll add comments inside the next block to describe each of these cases.
// Build a map of the namespaces for each FeatureVersion in our parent layer.
3f49fa4
to
dcf34cf
Compare
dcf34cf
to
5080cd4
Compare
PTAL 🎯 |
LGTM |
5080cd4
to
228cee7
Compare
228cee7
to
836d37b
Compare
No description provided.