gerlowskija commented on a change in pull request #302:
URL: https://github.com/apache/solr-operator/pull/302#discussion_r694262621
##########
File path: api/v1beta1/solrbackup_types.go
##########
@@ -38,12 +38,16 @@ type SolrBackupSpec struct {
// A reference to the SolrCloud to create a backup for
SolrCloud string `json:"solrCloud"`
+ // The name of the repository to use for the backup. Defaults to
"legacy_local_repository" if not specified (the
+ // auto-configured repository for legacy singleton volumes).
+ RepositoryName string `json:"repositoryName,omitempty"`
+
// The list of collections to backup. If empty, all collections in the
cloud will be backed up.
// +optional
Collections []string `json:"collections,omitempty"`
// Persistence is the specification on how to persist the backup data.
- Persistence PersistenceSource `json:"persistence"`
+ Persistence PersistenceSource `json:"persistence,omitempty"`
Review comment:
(I hadn't seen this "Suggested Change" feature of Github's - very cool!)
I'm not against having a pointer here - I'll have to read the docs a bit on
what you mean by "more optional" haha, but sounds good to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
