Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement pipeline state mechanism #59

Merged
merged 9 commits into from
Feb 24, 2020

Conversation

anupdhml
Copy link
Contributor

@anupdhml anupdhml commented Feb 22, 2020

@codecov
Copy link

codecov bot commented Feb 22, 2020

Codecov Report

Merging #59 into master will increase coverage by 0.12%.
The diff coverage is 88.34%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage   68.35%   68.47%   +0.12%     
==========================================
  Files         120      121       +1     
  Lines       13408    13499      +91     
==========================================
+ Hits         9165     9244      +79     
- Misses       4243     4255      +12
Impacted Files Coverage Δ
tests/query.rs 100% <ø> (ø) ⬆️
tremor-script/src/lib.rs 88.14% <ø> (ø) ⬆️
tremor-pipeline/src/op.rs 30% <0%> (-3.34%) ⬇️
tremor-pipeline/src/op/trickle/operator.rs 17.47% <100%> (+1.78%) ⬆️
tests/script_runtime_error.rs 93.75% <100%> (+0.2%) ⬆️
tremor-pipeline/src/op/debug/history.rs 76.66% <100%> (+0.39%) ⬆️
tremor-pipeline/src/op/generic/batch.rs 89.11% <100%> (+0.47%) ⬆️
tremor-pipeline/src/op/grouper/bucket.rs 54.43% <100%> (ø) ⬆️
tremor-script/src/ast/raw.rs 86.46% <100%> (+0.11%) ⬆️
tremor-pipeline/src/op/runtime/tremor.rs 74.28% <100%> (+0.75%) ⬆️
... and 18 more

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 864e776...00ac5d8. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Feb 24, 2020

Codecov Report

Merging #59 into master will increase coverage by 0.14%.
The diff coverage is 89.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage    66.5%   66.64%   +0.14%     
==========================================
  Files         122      123       +1     
  Lines       13931    14020      +89     
==========================================
+ Hits         9265     9344      +79     
- Misses       4666     4676      +10
Impacted Files Coverage Δ
tests/query.rs 100% <ø> (ø) ⬆️
tremor-script/src/lib.rs 88.14% <ø> (ø) ⬆️
tremor-pipeline/src/op.rs 33.33% <ø> (ø) ⬆️
tremor-pipeline/src/op/trickle/operator.rs 17.47% <100%> (+1.78%) ⬆️
tests/script_runtime_error.rs 93.75% <100%> (+0.2%) ⬆️
tremor-pipeline/src/op/debug/history.rs 76.66% <100%> (+0.39%) ⬆️
tremor-pipeline/src/op/generic/batch.rs 89.11% <100%> (+0.47%) ⬆️
tremor-pipeline/src/op/grouper/bucket.rs 54.43% <100%> (ø) ⬆️
tremor-script/src/ast/raw.rs 86.71% <100%> (+0.11%) ⬆️
tremor-pipeline/src/op/runtime/tremor.rs 74.28% <100%> (+0.75%) ⬆️
... and 18 more

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 8b607e9...05fa93b. Read the comment docs.

Licenser
Licenser previously approved these changes Feb 24, 2020
anupdhml and others added 5 commits February 24, 2020 10:43
Adds a new counter operator that utilizes the functionality.
Also introduces "state" as a keyword in the language.
Remove on_event2 and consolidate to on_event for state mechanism

Cleanup CIDR tremor script test

Support state mechanism in tremor-tool script run cmd

Fix broken tests and add query state test
Also explain trickery done for state assignments
@anupdhml anupdhml force-pushed the rfc_0002_pipeline_state_mechanism branch from 93fd74a to b3fdb3f Compare February 24, 2020 10:37
@anupdhml anupdhml force-pushed the rfc_0002_pipeline_state_mechanism branch from 2e8a495 to 5c559b7 Compare February 24, 2020 11:27
Licenser
Licenser previously approved these changes Feb 24, 2020
@Licenser Licenser merged commit 1049ad4 into master Feb 24, 2020
@Licenser Licenser deleted the rfc_0002_pipeline_state_mechanism branch February 24, 2020 12:16
anupdhml added a commit to tremor-rs/tremor-vim that referenced this pull request Feb 26, 2020
Small follow-up from state mechanism implementation in:
tremor-rs/tremor-runtime#59
anupdhml added a commit to tremor-rs/tremor-vscode that referenced this pull request Feb 26, 2020
Small follow-up from state mechanism implementation in:
tremor-rs/tremor-runtime#59
anupdhml added a commit to tremor-rs/tremor-mkdocs-lexer that referenced this pull request Feb 26, 2020
Small follow-up from state mechanism implementation in:
tremor-rs/tremor-runtime#59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants