search

tremor-rs / tremor-runtime

Main Tremor Project Rust Codebase
https://www.tremor.rs
Apache License 2.0
842 stars 127 forks source link

Polish tremor-runtime API for better embeddability #2575

Closed Licenser closed 3 weeks ago

Licenser commented 3 weeks ago

Signed-off-by: Heinz N. Gies heinz@licenser.net

Pull request

Description

Publish nessessary API from tremor-runtime to allow embedding the runtime in other applications.

Also polish some of the public interface.

Checklist

Performance

-/-

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 92.64706% with 20 lines in your changes missing coverage. Please review.

Project coverage is 91.35%. Comparing base (81f32a3) to head (03d246e).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/graphs/tree.svg?width=650&height=150&src=pr&token=d1bhuZGcOK&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs)](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) ```diff @@ Coverage Diff @@ ## main #2575 +/- ## ========================================== - Coverage 91.35% 91.35% -0.01% ========================================== Files 301 302 +1 Lines 58936 59159 +223 ========================================== + Hits 53843 54043 +200 - Misses 5093 5116 +23 ``` | [Flag](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | Coverage Δ | | |---|---|---| | [e2e-command](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `11.36% <0.00%> (-0.04%)` | :arrow_down: | | [e2e-integration](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `52.29% <39.26%> (-0.25%)` | :arrow_down: | | [e2e-unit](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `12.60% <0.00%> (-0.04%)` | :arrow_down: | | [e2etests](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `54.64% <39.26%> (-0.25%)` | :arrow_down: | | [tremorapi](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `14.49% <40.71%> (+0.03%)` | :arrow_up: | | [tremorcodec](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `63.04% <ø> (ø)` | | | [tremorcommon](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `61.84% <100.00%> (ø)` | | | [tremorconnectors](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `28.60% <1.29%> (-0.03%)` | :arrow_down: | | [tremorconnectorsaws](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `11.49% <0.00%> (-0.01%)` | :arrow_down: | | [tremorconnectorsgcp](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `21.29% <0.00%> (-0.03%)` | :arrow_down: | | [tremorconnectorsobjectstorage](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `0.00% <ø> (ø)` | | | [tremorconnectorsotel](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `12.68% <0.00%> (-0.03%)` | :arrow_down: | | [tremorconnectorstesthelpers](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `60.56% <ø> (ø)` | | | [tremorinflux](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `87.72% <ø> (ø)` | | | [tremorinterceptor](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `53.84% <ø> (ø)` | | | [tremorpipeline](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `31.07% <0.00%> (-0.08%)` | :arrow_down: | | [tremorruntime](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `47.18% <91.63%> (+0.22%)` | :arrow_up: | | [tremorscript](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `54.28% <ø> (ø)` | | | [tremorsystem](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `5.75% <85.71%> (+0.32%)` | :arrow_up: | | [tremorvalue](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `70.42% <ø> (-0.04%)` | :arrow_down: | | [unittests](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | `88.28% <92.64%> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | Coverage Δ | | |---|---|---| | [src/lib.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=src%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-c3JjL2xpYi5ycw==) | `100.00% <100.00%> (ø)` | | | [tremor-api/src/api.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-api%2Fsrc%2Fapi.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWFwaS9zcmMvYXBpLnJz) | `90.93% <100.00%> (ø)` | | | [tremor-archive/src/lib.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-archive%2Fsrc%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWFyY2hpdmUvc3JjL2xpYi5ycw==) | `84.03% <100.00%> (+0.22%)` | :arrow_up: | | [tremor-common/src/alias.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-common%2Fsrc%2Falias.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWNvbW1vbi9zcmMvYWxpYXMucnM=) | `89.90% <100.00%> (ø)` | | | [tremor-connectors/src/impls/bench.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-connectors%2Fsrc%2Fimpls%2Fbench.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWNvbm5lY3RvcnMvc3JjL2ltcGxzL2JlbmNoLnJz) | `89.27% <100.00%> (+0.09%)` | :arrow_up: | | [tremor-connectors/src/impls/cb.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-connectors%2Fsrc%2Fimpls%2Fcb.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWNvbm5lY3RvcnMvc3JjL2ltcGxzL2NiLnJz) | `94.59% <100.00%> (+0.07%)` | :arrow_up: | | [tremor-connectors/src/impls/exit.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-connectors%2Fsrc%2Fimpls%2Fexit.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWNvbm5lY3RvcnMvc3JjL2ltcGxzL2V4aXQucnM=) | `100.00% <100.00%> (ø)` | | | [tremor-connectors/src/impls/http/client.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-connectors%2Fsrc%2Fimpls%2Fhttp%2Fclient.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLWNvbm5lY3RvcnMvc3JjL2ltcGxzL2h0dHAvY2xpZW50LnJz) | `96.52% <100.00%> (ø)` | | | [tremor-system/src/lib.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=tremor-system%2Fsrc%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-dHJlbW9yLXN5c3RlbS9zcmMvbGliLnJz) | `100.00% <ø> (ø)` | | | [src/system.rs](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree&filepath=src%2Fsystem.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs#diff-c3JjL3N5c3RlbS5ycw==) | `94.27% <98.59%> (+9.89%)` | :arrow_up: | | ... and [2 more](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) | | ... and [6 files with indirect coverage changes](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs). Last update [81f32a3...03d246e](https://app.codecov.io/gh/tremor-rs/tremor-runtime/pull/2575?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tremor-rs).
mfelsche commented 3 weeks ago

:poland:

Licenser commented 3 weeks ago

Ja clippy and coverage, fixing that right now :)