Merge pull request grafana#1100 from cortexproject/just-one-nonperiodic

more table-manager fixes
cyriltovena · Nov 1, 2018 · c6f8c3d · c6f8c3d
2 parents 1cc9cb9 + 4bf217c
commit c6f8c3d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/schema_config.go b/schema_config.go
@@ -98,6 +98,7 @@ func (cfg *SchemaConfig) translate() error {
 			Store:  cfg.legacy.StorageClient,
 			IndexTables: PeriodicTableConfig{
 				Prefix: cfg.legacy.OriginalTableName,
+				Tags:   cfg.legacy.IndexTables.Tags,
 			},
 		})
 	}

diff --git a/table_manager.go b/table_manager.go
@@ -189,6 +189,10 @@ func (m *TableManager) calculateExpectedTables() []TableDesc {
 			continue
 		}
 		if config.IndexTables.Period == 0 { // non-periodic table
+			if len(result) > 0 && result[len(result)-1].Name == config.IndexTables.Prefix {
+				continue // already got a non-periodic table with this name
+			}
+
 			table := TableDesc{
 				Name:             config.IndexTables.Prefix,
 				ProvisionedRead:  config.IndexTables.InactiveReadThroughput,