Fix: “pointer to empty string” is not omitted bug of AWS X-Ray Exporter

[EdZou]

Description:
current exporter is broken because of #691 - unfortunately omitempty doesn’t work because “pointer to empty string” is not omitted.

Now added a util function under "internal/common/awsxray" repository and take use of this to check the "pointer to empty string"

If you would like to check @anuraaga

[pjanotti]

Hi @EdZou - the change itself looks fine but could you please add a test to cover this case?

/cc @kbrockhoff @anuraaga

[EdZou]

Hello @pjanotti I just added unit test, if you would like to review
cc: @kbrockhoff @anuraaga

[codecov]

Codecov Report

Merging #830 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #830   +/-   ##
=======================================
  Coverage   88.16%   88.16%           
=======================================
  Files         231      232    +1     
  Lines       12408    12412    +4     
=======================================
+ Hits        10939    10943    +4     
  Misses       1114     1114           
  Partials      355      355           

Flag	Coverage Δ
#integration	72.00% <ø> (ø)
#unit	87.98% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
exporter/awsxrayexporter/translator/aws.go	96.74% <100.00%> (ø)
exporter/awsxrayexporter/translator/http.go	94.11% <100.00%> (ø)
exporter/awsxrayexporter/translator/segment.go	93.25% <100.00%> (ø)
exporter/awsxrayexporter/translator/service.go	100.00% <100.00%> (ø)
exporter/awsxrayexporter/translator/sql.go	83.60% <100.00%> (ø)
internal/common/awsxray/util.go	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02f7291...1776559. Read the comment docs.

[anuraaga]

Thanks, looks like there's a small build failure and one doc comment, but otherwise LGTM

[anuraaga]

Suggested change

	// Util to check nil string pointer
	// String returns a pointer to the provided string, or nil if it is an empty string.

[anuraaga]

Looks like just a flaky loadtest - @pjanotti @open-telemetry/collector-maintainers do you mind rerunning it and merging if things look ok? Thanks!