Skip to content

Commit

Permalink
Merge pull request #420 from bruin-data/patch/fix-metadata-push-for-b…
Browse files Browse the repository at this point in the history
…q-sensors

fix metadata push for bq sensors
  • Loading branch information
karakanb authored Jan 25, 2025
2 parents 0ecd685 + 6424d3a commit 023b98e
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 19 deletions.
8 changes: 8 additions & 0 deletions cmd/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -638,6 +638,14 @@ func setupExecutors(
mainExecutors[pipeline.AssetTypeBigquerySource][scheduler.TaskInstanceTypeColumnCheck] = bqCheckRunner
mainExecutors[pipeline.AssetTypeBigquerySource][scheduler.TaskInstanceTypeCustomCheck] = customCheckRunner

mainExecutors[pipeline.AssetTypeBigqueryTableSensor][scheduler.TaskInstanceTypeMetadataPush] = metadataPushOperator
mainExecutors[pipeline.AssetTypeBigqueryTableSensor][scheduler.TaskInstanceTypeColumnCheck] = bqCheckRunner
mainExecutors[pipeline.AssetTypeBigqueryTableSensor][scheduler.TaskInstanceTypeCustomCheck] = customCheckRunner

mainExecutors[pipeline.AssetTypeBigqueryQuerySensor][scheduler.TaskInstanceTypeMetadataPush] = metadataPushOperator
mainExecutors[pipeline.AssetTypeBigqueryQuerySensor][scheduler.TaskInstanceTypeColumnCheck] = bqCheckRunner
mainExecutors[pipeline.AssetTypeBigqueryQuerySensor][scheduler.TaskInstanceTypeCustomCheck] = customCheckRunner

mainExecutors[pipeline.AssetTypeBigquerySeed][scheduler.TaskInstanceTypeMain] = seedOperator
mainExecutors[pipeline.AssetTypeBigquerySeed][scheduler.TaskInstanceTypeColumnCheck] = bqCheckRunner
mainExecutors[pipeline.AssetTypeBigquerySeed][scheduler.TaskInstanceTypeCustomCheck] = customCheckRunner
Expand Down
31 changes: 12 additions & 19 deletions pkg/executor/defaults.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,22 @@ type Config map[scheduler.TaskInstanceType]Operator

var DefaultExecutorsV2 = map[pipeline.AssetType]Config{
pipeline.AssetTypeBigqueryQuery: {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeColumnCheck: NoOpOperator{},
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMetadataPush: NoOpOperator{},
scheduler.TaskInstanceTypeColumnCheck: NoOpOperator{},
scheduler.TaskInstanceTypeCustomCheck: NoOpOperator{},
},
pipeline.AssetTypeBigqueryTableSensor: {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMetadataPush: NoOpOperator{},
scheduler.TaskInstanceTypeColumnCheck: NoOpOperator{},
scheduler.TaskInstanceTypeCustomCheck: NoOpOperator{},
},
pipeline.AssetTypeBigqueryQuerySensor: {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
"bq.cost_tracker": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
"bq.transfer": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
"bq.sensor.partition": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMetadataPush: NoOpOperator{},
scheduler.TaskInstanceTypeColumnCheck: NoOpOperator{},
scheduler.TaskInstanceTypeCustomCheck: NoOpOperator{},
},
pipeline.AssetTypeBigquerySource: {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
Expand All @@ -37,12 +36,6 @@ var DefaultExecutorsV2 = map[pipeline.AssetType]Config{
scheduler.TaskInstanceTypeMain: NoOpOperator{},
scheduler.TaskInstanceTypeMetadataPush: NoOpOperator{},
},
"gcs.from.s3": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
"gcs.delete": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
"gcs.sensor.object_sensor_with_prefix": {
scheduler.TaskInstanceTypeMain: NoOpOperator{},
},
Expand Down

0 comments on commit 023b98e

Please sign in to comment.