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

Update RedisTimeSeries.java #22

Merged

Conversation

pariskeras
Copy link
Contributor

Supporting functionality of TSADD from RedisTimeseries Documentation of commands
TS.ADD: use * for automatic timestamp (using the system clock)

Supporting functionality of TSADD from RedisTimeseries Documentation of commands
TS.ADD: use * for automatic timestamp (using the system clock)
@dengliming dengliming added enhancement New feature or request RedisTimeSeries labels May 27, 2021
@codecov
Copy link

codecov bot commented May 27, 2021

Codecov Report

Merging #22 (456bdd1) into master (34aadcb) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #22      +/-   ##
============================================
- Coverage     65.38%   65.33%   -0.05%     
  Complexity      574      574              
============================================
  Files           104      104              
  Lines          2051     2051              
  Branches        170      171       +1     
============================================
- Hits           1341     1340       -1     
  Misses          608      608              
- Partials        102      103       +1     
Impacted Files Coverage Δ
...g/redismodule/redistimeseries/RedisTimeSeries.java 88.29% <0.00%> (-1.07%) ⬇️

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 34aadcb...456bdd1. Read the comment docs.

Copy link
Owner

@dengliming dengliming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dengliming dengliming merged commit 89d5562 into dengliming:master May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request RedisTimeSeries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants