grouper/enricher produces/consumes keyed input

[laxmanchekka]

Description

grouper produces keyed output
enricher produces keyed input

Testing

Unit tests

Checklist:

• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• Any dependent changes have been merged and published in downstream modules

Documentation

NA

[codecov]

Codecov Report

Merging #314 (feae75d) into main (ef63989) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main     #314   +/-   ##
=========================================
  Coverage     79.94%   79.94%           
  Complexity     1365     1365           
=========================================
  Files           125      125           
  Lines          5384     5384           
  Branches        489      489           
=========================================
  Hits           4304     4304           
  Misses          857      857           
  Partials        223      223           

Flag	Coverage Δ
unit	79.94% <100.00%> (ø)

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

Impacted Files	Coverage Δ
...trace/enricher/StructuredTraceEnrichProcessor.java	86.84% <ø> (ø)
...rtrace/core/rawspansgrouper/RawSpansProcessor.java	81.57% <ø> (ø)
...ce/traceenricher/trace/enricher/TraceEnricher.java	72.72% <100.00%> (ø)
...pertrace/core/rawspansgrouper/RawSpansGrouper.java	90.00% <100.00%> (ø)
...race/core/rawspansgrouper/TraceEmitPunctuator.java	75.43% <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 ef63989...feae75d. Read the comment docs.

[kotharironak]

lgtm

Can you check snyk issues?

[kotharironak]

Is it possible to be specific? Or was this auto-suggested of replacing to * import?

[laxmanchekka]

done by spotless I guess. it prefers wildcard when number of classes used are more than a threshold

[saxenakshitiz]

Must be done by IDE. AFAIK spotless doesn't make this change.

[findingrish]

I guess we need to set a high number like 999 as import class count to be optimised with * in intellij.

[laxmanchekka]

lgtm

Can you check snyk issues?
@kotharironak : vulnerability is from data-model. we need to fix it separately and upgrade the lib here.

[github-actions]

Unit Test Results

  76 files  ±0    76 suites  ±0   1m 4s ⏱️ +4s
408 tests ±0  408 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a4e8fa4. ± Comparison against base commit ef63989.