Remove sdk/export/metric module, embed into the sdk/metric

[bogdandrutu]

Updates #2273.

Signed-off-by: Bogdan Drutu [email protected]

[codecov]

Codecov Report

Merging #2382 (3f91b9e) into main (aedef05) will increase coverage by 0.3%.
The diff coverage is 91.0%.

❗ Current head 3f91b9e differs from pull request most recent head 03f3482. Consider uploading reports for the commit 03f3482 to get more accurate results

@@           Coverage Diff           @@
##            main   #2382     +/-   ##
=======================================
+ Coverage   76.0%   76.3%   +0.3%     
=======================================
  Files        173     173             
  Lines      12010   12010             
=======================================
+ Hits        9132    9170     +38     
+ Misses      2633    2595     -38     
  Partials     245     245             

Impacted Files	Coverage Δ
bridge/opencensus/aggregation.go	95.7% <ø> (ø)
bridge/opencensus/exporter.go	100.0% <ø> (ø)
...otlp/otlpmetric/internal/metrictransform/metric.go	78.1% <ø> (ø)
...rs/otlp/otlpmetric/internal/otlpmetrictest/data.go	0.0% <0.0%> (ø)
...tlp/otlpmetric/internal/otlpmetrictest/otlptest.go	0.0% <ø> (ø)
exporters/otlp/otlpmetric/options.go	100.0% <ø> (ø)
exporters/prometheus/prometheus.go	65.2% <ø> (ø)
exporters/stdout/stdoutmetric/exporter.go	62.5% <ø> (ø)
sdk/metric/aggregator/aggregator.go	100.0% <ø> (ø)
sdk/metric/aggregator/aggregatortest/test.go	81.0% <ø> (ø)
... and 20 more

[MrAlias]

Please deprecate the old package instead of removing so users know what to use instead. We can remove in the following release to the release that includes this deprecation.

[jmacd]

I'd like to merge #2348 #2350 and #2352 before this sort of improvement. 😁

[jmacd]

See #2273.

[MrAlias]

@bogdandrutu all the dependent PRs have merged. Are you able to update this PR to deprecate the package?

[bogdandrutu]

@MrAlias deprecation is done.

[bogdandrutu]

@jmacd this is an intermediate PR, will do the last move inside the new package after. I have enough time until the next release. Is that ok with you @MrAlias ?

[MrAlias]

@jmacd this is an intermediate PR, will do the last move inside the new package after. I have enough time until the next release. Is that ok with you @MrAlias ?

SGTM