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

Expose API to check whether the Siddhi App is stateful or not #1373 #1413

Merged
merged 3 commits into from
Jul 24, 2019

Conversation

suhothayan
Copy link
Contributor

@suhothayan suhothayan commented Jul 24, 2019

Purpose

This exposes queries, partitions, windows, triggers, and tables to check whether they are stateful or not.
Also add more test cases that are missed before.
Fixes #1373

…io#1373

This expose queries, partitions, windows, triggers, and tables to check whether they are stateful or not.
@pcnfernando
Copy link
Contributor

I'm checking the Travis failure.

@codecov-io
Copy link

Codecov Report

Merging #1413 into master will increase coverage by 1.42%.
The diff coverage is 68.6%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1413      +/-   ##
============================================
+ Coverage     65.31%   66.73%   +1.42%     
  Complexity       37       37              
============================================
  Files           632      633       +1     
  Lines         26662    26688      +26     
  Branches       4214     4214              
============================================
+ Hits          17414    17810     +396     
+ Misses         7795     7401     -394     
- Partials       1453     1477      +24
Impacted Files Coverage Δ Complexity Δ
...core/src/main/java/io/siddhi/core/table/Table.java 28.01% <ø> (ø) 0 <0> (ø) ⬇️
...ore/util/parser/helper/DefinitionParserHelper.java 74.42% <ø> (ø) 0 <0> (ø) ⬇️
...n/java/io/siddhi/core/stream/output/sink/Sink.java 72.18% <0%> (-0.55%) 0 <0> (ø)
...ava/io/siddhi/core/stream/input/source/Source.java 82.35% <0%> (-1.23%) 0 <0> (ø)
...va/io/siddhi/core/util/parser/SiddhiAppParser.java 78.27% <100%> (+0.08%) 0 <0> (ø) ⬇️
...siddhi/core/partition/PartitionStreamReceiver.java 77.04% <100%> (ø) 0 <0> (ø) ⬇️
...n/java/io/siddhi/core/util/parser/QueryParser.java 94.59% <100%> (ø) 0 <0> (ø) ⬇️
...re/src/main/java/io/siddhi/core/window/Window.java 76.36% <100%> (+0.21%) 0 <0> (ø) ⬇️
.../main/java/io/siddhi/core/table/InMemoryTable.java 98.64% <100%> (+6.86%) 0 <0> (ø) ⬇️
...uery/processor/stream/AbstractStreamProcessor.java 97.91% <100%> (+0.04%) 0 <0> (ø) ⬇️
... and 35 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 548c59a...659f30c. Read the comment docs.

@pcnfernando
Copy link
Contributor

Travis failed due to an intermittent test failure.

@pcnfernando pcnfernando merged commit 9e28eb5 into siddhi-io:master Jul 24, 2019
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.

Expose an api to check whether a compiled Siddhi app is stateful or not
3 participants