splitTimeRangeExt is considered harmful

platform/optimize/pipeline.go

@@ -27,19 +27,13 @@ func NewOptimizePipeline(config *config.QuesmaConfiguration) model.QueryTransfor
 	// TODO remove this line when splitTimeRange is removed
 	// this is just to satisfy the linter
 	_ = &splitTimeRange{}
+	_ = &splitTimeRangeExt{}
 	return &OptimizePipeline{
 		config: config,
 		optimizations: []OptimizeTransformer{
 			&truncateDate{truncateTo: 5 * time.Minute},
 			&cacheQueries{},
 			&materializedViewReplace{},
-			// TODO finally remove this transformer
-			// commenting out splitTimeRange for now
-			// as we have splitTimeRangeExt that uses novel approach
-			// of splitting queries based on time range
-			// executing them in parallel and finally merging results
-			// &splitTimeRange{},
-			&splitTimeRangeExt{},
 		},
 	}
 }