Small comment cleanups

Author: shanson7

api/routes.go

@@ -55,7 +55,7 @@ func (s *Server) RegisterRoutes() {
 		ctx.Write(nil)
 	})
 
-	// Miscellaneous MetricTank-only user facing endpoints
+	// Miscellaneous Metrictank-only user facing endpoints
 	r.Combo("/showplan", cBody, withOrg, ready, bind(models.GraphiteRender{})).Get(s.showPlan).Post(s.showPlan)
 	r.Combo("/tags/terms", ready, bind(models.GraphiteTagTerms{})).Get(s.graphiteTagTerms).Post(s.graphiteTagTerms)
 	r.Combo("/ccache/delete", bind(models.CCacheDelete{})).Post(s.ccacheDelete).Get(s.ccacheDelete)

idx/idx.go

@@ -102,7 +102,7 @@ type MetricIndex interface {
 	// that duplicate entries will be returned.
 	FindByTag(orgId uint32, query tagquery.Query) []Node
 
-	// FindTerms tags a query object and executes the query on the index. The query
+	// FindTerms takes a query object and executes the query on the index. The query
 	// is composed of one or many query expressions. From the matching series, a count
 	// is kept for each value of the requested tags.
 	// The series are not deduplicated and in certain cases it is possible that some

idx/memory/memory.go

@@ -1116,8 +1116,6 @@ func (m *UnpartitionedMemoryIdx) FindTerms(orgID uint32, tags []string, query ta
 		return 0, nil
 	}
 
-	queryCtx := NewTagQueryContext(query)
-
 	needsName := false
 	terms := make(map[string]map[string]uint32)
 	for _, tag := range tags {
@@ -1127,11 +1125,11 @@ func (m *UnpartitionedMemoryIdx) FindTerms(orgID uint32, tags []string, query ta
 		terms[tag] = make(map[string]uint32)
 	}
 
+	queryCtx := NewTagQueryContext(query)
+
 	m.RLock()
 	defer m.RUnlock()
 
-	// TODO - terms for meta tags?
-
 	// construct the output slice of idx.Node's such that there is only 1 idx.Node for each path
 	resCh := m.idsByTagQuery(orgID, queryCtx)