Elasticsearch exporter dedot and dedup config has no effect

[CarsonCook]

Component(s)

exporter/elasticsearch

What happened?

Description

mapping.dedup and mapping.dedot configuration have no effect, they are always enabled and disabled, respectively:

// TODO: Apply encoding and field mapping settings.
model := &encodeModel{dedup: true, dedot: false}

https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/elasticsearchexporter/logs_exporter.go#L65

https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/elasticsearchexporter/trace_exporter.go#L61

Steps to Reproduce

1. Set mapping.dedot to true in the receiver's config.
2. Run and see dots are preserved

Expected Result

mapping.dedup and mapping.dedot config values are used.

Alternatively, change documentation to reflect these fields are not operational.

Actual Result

Fields have no effect.

Collector version

0.71.0

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")

OpenTelemetry Collector configuration

exporters:
  elasticsearch:
    mappings:
        dedot: true

Log output

No response

Additional context

PR fixing bug: #19420

[github-actions]

Pinging code owners:

• exporter/elasticsearch: @JaredTan95

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• exporter/elasticsearch: @JaredTan95

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been closed as inactive because it has been stale for 120 days with no activity.