Skip to content

fix bug caused by typo in "WindowFlux"#108

Merged
bednar merged 2 commits intoinfluxdata:masterfrom
jay-w-opus:patch-1
Apr 27, 2020
Merged

fix bug caused by typo in "WindowFlux"#108
bednar merged 2 commits intoinfluxdata:masterfrom
jay-w-opus:patch-1

Conversation

@jay-w-opus
Copy link
Copy Markdown
Contributor

@jay-w-opus jay-w-opus commented Apr 26, 2020

Closes #

I'm not sure if I miss anything. I was trying to put "offset" into the function but only found "start". However, according to the spec, "start" does not seem to be an acceptable argument, according to https://github.com/influxdata/flux/blob/master/execute/window.go

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Sign CLA (if not already signed)

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #108 into master will decrease coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #108      +/-   ##
============================================
- Coverage     90.09%   90.00%   -0.10%     
  Complexity      429      429              
============================================
  Files           136      136              
  Lines          5392     5392              
  Branches        218      218              
============================================
- Hits           4858     4853       -5     
- Misses          446      448       +2     
- Partials         88       91       +3     
Impacted Files Coverage Δ Complexity Δ
...a/com/influxdb/query/dsl/functions/WindowFlux.java 100.00% <100.00%> (ø) 12.00 <1.00> (ø)
.../influxdb/client/internal/AbstractWriteClient.java 89.57% <0.00%> (-1.90%) 0.00% <0.00%> (ø%)
...luxdb/client/kotlin/internal/QueryKotlinApiImpl.kt 76.11% <0.00%> (-1.50%) 0.00% <0.00%> (ø%)

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 a6101b5...466df11. Read the comment docs.

@bednar bednar self-requested a review April 27, 2020 07:27
@bednar bednar merged commit 7e122e8 into influxdata:master Apr 27, 2020
@bednar bednar added this to the 1.8.0 milestone Apr 27, 2020
@bednar bednar added the bug Something isn't working label Apr 27, 2020
@jay-w-opus jay-w-opus deleted the patch-1 branch April 30, 2020 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants