Skip to content

feat: flux query profiler #260

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

Merged
merged 9 commits into from
May 28, 2021
Merged

feat: flux query profiler #260

merged 9 commits into from
May 28, 2021

Conversation

rhajek
Copy link
Contributor

@rhajek rhajek commented May 24, 2021

Proposed Changes

This PR adds support for Flux Query profiler - https://docs.influxdata.com/influxdb/v2.0/reference/flux/stdlib/profiler/

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • pytest tests completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@rhajek rhajek changed the title Feat/profiler query feat: flux query profiler May 24, 2021
@rhajek rhajek requested a review from bednar May 25, 2021 07:32
@rhajek rhajek requested a review from bednar May 27, 2021 14:27
@codecov-commenter
Copy link

codecov-commenter commented May 28, 2021

Codecov Report

Merging #260 (2089f71) into master (30b75c9) will increase coverage by 0.49%.
The diff coverage is 98.80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
+ Coverage   91.91%   92.41%   +0.49%     
==========================================
  Files          25       25              
  Lines        2042     2109      +67     
==========================================
+ Hits         1877     1949      +72     
+ Misses        165      160       -5     
Impacted Files Coverage Δ
influxdb_client/client/influxdb_client.py 98.08% <90.90%> (-0.57%) ⬇️
influxdb_client/client/flux_csv_parser.py 97.48% <100.00%> (+1.16%) ⬆️
influxdb_client/client/query_api.py 99.09% <100.00%> (+1.37%) ⬆️
influxdb_client/client/flux_table.py 97.77% <0.00%> (+8.88%) ⬆️

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 30b75c9...2089f71. Read the comment docs.

@rhajek rhajek force-pushed the feat/profiler_query branch from 33caaf4 to 6fd03d8 Compare May 28, 2021 06:42
@rhajek rhajek requested a review from bednar May 28, 2021 07:26
Co-authored-by: Jakub Bednář <jakub.bednar@gmail.com>
@rhajek rhajek merged commit 3235d4e into master May 28, 2021
@rhajek rhajek deleted the feat/profiler_query branch May 28, 2021 07:50
@bednar bednar added this to the 1.18.0 milestone May 28, 2021
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy