Skip to content
This repository was archived by the owner on Aug 30, 2025. It is now read-only.

Commit e42ad34

Browse files
committed
udpates dst config builder
1 parent 53559b2 commit e42ad34

1 file changed

Lines changed: 11 additions & 1 deletion

File tree

  • internal/benthos/benthos-builder/builders

internal/benthos/benthos-builder/builders/sql.go

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import (
1212
sqlmanager_mssql "github.com/nucleuscloud/neosync/backend/pkg/sqlmanager/mssql"
1313
sqlmanager_shared "github.com/nucleuscloud/neosync/backend/pkg/sqlmanager/shared"
1414
cascade_settings "github.com/nucleuscloud/neosync/internal/benthos/benthos-builder/builders/jobmapping-builder/settings"
15+
jobmapping_builder_shared "github.com/nucleuscloud/neosync/internal/benthos/benthos-builder/builders/jobmapping-builder/shared"
1516
jobmapping_builder_sql "github.com/nucleuscloud/neosync/internal/benthos/benthos-builder/builders/jobmapping-builder/sql"
1617
bb_internal "github.com/nucleuscloud/neosync/internal/benthos/benthos-builder/internal"
1718
bb_shared "github.com/nucleuscloud/neosync/internal/benthos/benthos-builder/shared"
@@ -362,6 +363,15 @@ func (b *sqlSyncBuilder) BuildDestinationConfig(
362363
params *bb_internal.DestinationParams,
363364
) (*bb_internal.BenthosDestinationConfig, error) {
364365
logger := params.Logger
366+
// need to do this due to the CLI not actually calling BuildSource
367+
// So we need to conditionally use the job mappings from the builder for the CLI to work properly
368+
var jobMappings []*shared.JobTransformationMapping
369+
if len(b.jobMappings) > 0 {
370+
jobMappings = b.jobMappings
371+
} else {
372+
jobMappings = jobmapping_builder_shared.JobMappingsFromLegacyMappings(params.Job.GetMappings())
373+
}
374+
365375
benthosConfig := params.SourceConfig
366376
tableKey := neosync_benthos.BuildBenthosTable(
367377
benthosConfig.TableSchema,
@@ -406,7 +416,7 @@ func (b *sqlSyncBuilder) BuildDestinationConfig(
406416
colTransformerMap := b.colTransformerMap
407417
// lazy load
408418
if len(colTransformerMap) == 0 {
409-
groupedMappings := groupMappingsByTable(b.jobMappings)
419+
groupedMappings := groupMappingsByTable(jobMappings)
410420
groupedTableMapping := getTableMappingsMap(groupedMappings)
411421
colTMap := getColumnTransformerMap(
412422
groupedTableMapping,

0 commit comments

Comments
 (0)