Cleanup§

Author: trzysiek

platform/clickhouse/schema.go

@@ -6,7 +6,6 @@ import (
 	"fmt"
 	"github.com/QuesmaOrg/quesma/platform/logger"
 	"github.com/QuesmaOrg/quesma/platform/util"
-	"github.com/k0kubun/pp"
 	"math"
 	"reflect"
 	"strings"
@@ -339,7 +338,6 @@ func (col *Column) createTableString(indentLvl int) string {
 
 // TODO TTL only by timestamp for now!
 func (config *ChTableConfig) CreateTablePostFieldsString() string {
-	pp.Println("aaaa", config)
 	s := "ENGINE = " + config.Engine + "\n"
 	if config.OrderBy != "" {
 		s += "ORDER BY " + config.OrderBy + "\n"

platform/config/config.go

@@ -5,7 +5,6 @@ package config
 import (
 	"fmt"
 	"github.com/QuesmaOrg/quesma/platform/elasticsearch/elasticsearch_field_types"
-	"github.com/QuesmaOrg/quesma/platform/logger"
 	"github.com/QuesmaOrg/quesma/platform/util"
 	"github.com/hashicorp/go-multierror"
 	"github.com/knadh/koanf/parsers/yaml"
@@ -84,7 +83,6 @@ func loadConfigFile() {
 		configPath = defaultConfigFileName
 	}
 	fmt.Printf("Using config file: [%s]\n", configPath)
-	logger.Error().Msgf("Using config file: [%s]", configPath)
 	if _, err := os.Stat(configPath); os.IsNotExist(err) {
 		fmt.Printf("Error loading config file [%v], proceeding without it...\n", err)
 		return
@@ -194,7 +192,6 @@ func (c *QuesmaConfiguration) OptimizersConfigAsString() string {
 }
 
 func (c *QuesmaConfiguration) String() string {
-	logger.Error().Msgf("QuesmaConfiguration: %v", c.IndexConfig)
 	var indexConfigs string
 	for indexName, idx := range c.IndexConfig {
 		indexConfigs += idx.String(indexName)

platform/config/config_v2.go

@@ -955,7 +955,6 @@ func (c *QuesmaNewConfiguration) TranslateToLegacyConfig() QuesmaConfiguration {
 				errAcc = multierror.Append(errAcc, errTarget)
 			}
 			for _, target := range targets {
-				logger.Info().Msgf("target %v", target)
 				if targetType, found := c.getTargetType(target.target); found {
 					if !slices.Contains(processedConfig.IngestTarget, targetType) {
 						processedConfig.IngestTarget = append(processedConfig.IngestTarget, targetType)
@@ -972,11 +971,9 @@ func (c *QuesmaNewConfiguration) TranslateToLegacyConfig() QuesmaConfiguration {
 				if val, exists := target.properties["tableName"]; exists {
 					processedConfig.Override = val.(string)
 				}
-				if val, exists := target.properties["partitionBy"]; exists {
-					processedConfig.PartitionBy = val.(string)
-				}
 			}
 
+			// maybe delete this?
 			if indexConfig.PartitionBy != "" {
 				processedConfig.PartitionBy = indexConfig.PartitionBy
 			}

platform/config/index_config.go

@@ -4,7 +4,6 @@ package config
 
 import (
 	"fmt"
-	"github.com/QuesmaOrg/quesma/platform/logger"
 	"slices"
 	"strings"
 )
@@ -62,7 +61,6 @@ func (c IndexConfiguration) String(indexName string) string {
 	if c.UseCommonTable {
 		builder.WriteString(", useSingleTable: true")
 	}
-	logger.Error().Msgf("IndexConfiguration: part %s ", c.PartitionBy)
 	if len(c.PartitionBy) > 0 {
 		builder.WriteString(", partitionBy: " + c.PartitionBy)
 	}