Some small improvements

[mfelsche]

Pull request

Description

Related

• RFC
• Related Issues: fixes #000, closed #000
• Related docs PR

Checklist

• The RFC, if required, has been submitted and approved
• Any user-facing impact of the changes is reflected in docs.tremor.rs
• The code is tested
• Use of unsafe code is reasoned about in a comment
• Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
• The performance impact of the change is measured (see below)

Performance

[Licenser]

Nice catch!

[codecov]

Codecov Report

Merging #1619 (a7785f6) into main (80b7e34) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1619   +/-   ##
=======================================
  Coverage   85.61%   85.62%           
=======================================
  Files         232      232           
  Lines       47109    47109           
=======================================
+ Hits        40334    40338    +4     
+ Misses       6775     6771    -4     

Flag	Coverage Δ
unittests	85.62% <100.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
src/connectors/impls/metronome.rs	98.63% <100.00%> (+1.40%)	⬆️
src/system/flow_supervisor.rs	86.44% <100.00%> (+0.72%)	⬆️
src/connectors.rs	85.91% <0.00%> (-0.31%)	⬇️
src/pipeline.rs	76.66% <0.00%> (-0.28%)	⬇️
src/system/flow.rs	89.90% <0.00%> (-0.20%)	⬇️
tremor-value/src/value.rs	98.99% <0.00%> (+0.09%)	⬆️
src/connectors/source/channel_source.rs	80.00% <0.00%> (+1.66%)	⬆️
src/version.rs	87.50% <0.00%> (+12.50%)	⬆️

---

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 80b7e34...a7785f6. Read the comment docs.

[coveralls]

Pull Request Test Coverage Report for Build 2293068634

• 5 of 5 (100.0%) changed or added relevant lines in 2 files are covered.
• 88 unchanged lines in 4 files lost coverage.
• Overall coverage decreased (-0.06%) to 85.627%

---

Files with Coverage Reduction	New Missed Lines	%
src/system/flow.rs	1	89.9%
src/system/flow_supervisor.rs	1	86.44%
src/connectors.rs	2	85.91%
src/pipeline.rs	84	76.67%

Totals
Change from base Build 2292922964:	-0.06%
Covered Lines:	40338
Relevant Lines:	47109

---

💛 - Coveralls

[darach]

LGTM 👍