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.

Sign up for GitHub

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

Jump to bottom

ref #8 add TS.GET, TS.MGET, TS.ALTER and TS.QUERYINDEX #9

Merged
merged 9 commits into from
Jan 28, 2020
Merged

ref #8 add TS.GET, TS.MGET, TS.ALTER and TS.QUERYINDEX #9

merged 9 commits into from
Jan 28, 2020

Conversation

gkorland
Copy link
Contributor

@gkorland gkorland commented Jan 26, 2020 •
edited by filipecosta90
Loading

fixes #8

@codecov
Copy link

codecov bot commented Jan 26, 2020 •
edited
Loading

Codecov Report

Merging #9 into master will increase coverage by 0.51%.
The diff coverage is 78.78%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   87.64%   88.15%   +0.51%     
==========================================
  Files           9        9              
  Lines         348      380      +32     
  Branches       67       76       +9     
==========================================
+ Hits          305      335      +30     
+ Misses          9        6       -3     
- Partials       34       39       +5
Impacted Files Coverage Δ
...in/java/com/redislabs/redistimeseries/Command.java 100% <100%> (ø) ⬆️
...com/redislabs/redistimeseries/RedisTimeSeries.java 84.64% <76.66%> (+0.99%) ⬆️

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 e453812...18722c9. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Jan 26, 2020

This pull request introduces 1 alert when merging e453812 into ec7dd8c - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jan 26, 2020

This pull request introduces 1 alert when merging 945c16d into 4e5d7ce - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

filipecosta90
filipecosta90 previously approved these changes Jan 26, 2020
View reviewed changes
@filipecosta90 filipecosta90 mentioned this pull request Jan 27, 2020
Closed
@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2020

This pull request introduces 2 alerts when merging 1330588 into 4e5d7ce - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2020

This pull request introduces 2 alerts when merging 54a54c8 into 4e5d7ce - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2020

This pull request introduces 2 alerts when merging 38877c0 into 4e5d7ce - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

@gkorland gkorland marked this pull request as ready for review January 27, 2020 16:57
@filipecosta90 filipecosta90 changed the title ref #8 add TS.GET and TS.MGET ref #8 add TS.GET, TS.MGET, TS.ALTER and TS.QUERYINDEX Jan 28, 2020
@filipecosta90 filipecosta90 merged commit 6a39b9b into master Jan 28, 2020
@filipecosta90 filipecosta90 mentioned this pull request Jan 28, 2020
Closed
@gkorland gkorland deleted the mget branch January 28, 2020 10:40
@gkorland gkorland added the enhancement New feature or request label Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filipecosta90 filipecosta90 filipecosta90 approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

support for TS.GET and TS.MGET
2 participants
@gkorland @filipecosta90