Skip to content

Commit

Permalink
Also kill batch size
Browse files Browse the repository at this point in the history
  • Loading branch information
nathan-artie committed Jun 18, 2024
1 parent 61b5e43 commit 2884164
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
3 changes: 0 additions & 3 deletions clients/bigquery/bigquery.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ const (

type Store struct {
configMap *types.DwhToTablesConfigMap
batchSize int
config config.Config

db.Store
Expand Down Expand Up @@ -223,7 +222,6 @@ func (s *Store) Dedupe(tableID sql.TableIdentifier, primaryKeys []string, includ
}

func LoadBigQuery(cfg config.Config, _store *db.Store) (*Store, error) {
cfg.BigQuery.LoadDefaultValues()
if _store != nil {
// Used for tests.
return &Store{
Expand All @@ -250,7 +248,6 @@ func LoadBigQuery(cfg config.Config, _store *db.Store) (*Store, error) {
return &Store{
Store: store,
configMap: &types.DwhToTablesConfigMap{},
batchSize: cfg.BigQuery.BatchSize,
config: cfg,
}, nil
}
7 changes: 0 additions & 7 deletions lib/config/bigquery.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,6 @@ type BigQuery struct {
DefaultDataset string `yaml:"defaultDataset"`
ProjectID string `yaml:"projectID"`
Location string `yaml:"location"`
BatchSize int `yaml:"batchSize"`
}

func (b *BigQuery) LoadDefaultValues() {
if b.BatchSize == 0 {
b.BatchSize = 1000
}
}

// DSN - returns the notation for BigQuery following this format: bigquery://projectID/[location/]datasetID?queryString
Expand Down

0 comments on commit 2884164

Please sign in to comment.