updating comments

TykTechnologies · Aug 1, 2023 · 67584a5 · 67584a5
1 parent 98fdc10
commit 67584a5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/linter.yaml b/.github/workflows/linter.yaml
@@ -27,7 +27,6 @@ jobs:
       go: ${{ matrix.go }}
       redis: 5
       mongo: 4.2
-      test-options: "-count=1 -failfast -v"
   golangci:
     needs: gotest
     if: ${{ always() }}

diff --git a/pumps/sql_aggregate.go b/pumps/sql_aggregate.go
@@ -123,13 +123,14 @@ func (c *SQLAggregatePump) Init(conf interface{}) error {
 			return err
 		}
 
+		// we can run the index creation in background only for postgres since it supports CONCURRENTLY
 		shouldRunOnBackground := false
 		if c.dbType == "postgres" {
 			shouldRunOnBackground = true
 			c.backgroundIndexCreated = make(chan bool)
 		}
 
-		// if index doesn't exist, create it on background since it's going to migrate all the existing data
+		// if index doesn't exist, create it
 		if err := c.ensureIndex(analytics.AggregateSQLTable, shouldRunOnBackground); err != nil {
 			c.log.Error(err)
 			return err