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

docs: clarify how to use JMeter with InfluxDB 2 #588

Closed
wants to merge 1 commit into from

Conversation

bednar
Copy link
Contributor

@bednar bednar commented May 5, 2020

Description

Improved documentation how to use JMeter with InfluxDB 2.

Motivation and Context

Clarifying to user that the JMeter support InfluxDB 2 and how to configure JMeter for that.

How Has This Been Tested?

Verified the generated JavaDoc and documentation.

Screenshots (if appropriate):

component_reference.html#Backend_Listener

Before:

backend_listener

After:

backend_listener_new

realtime-results.html

Before:

real_time_results

After:

real_time_results_new

package-summary.html

Before:

package-summary

After:

package-summary_new

InfluxdbBackendListenerClient.html

Before:

InfluxdbBackendListenerClient

After:

InfluxdbBackendListenerClient_new

Types of changes

  • Documentation change

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.

@codecov-io
Copy link

@codecov-io codecov-io commented May 5, 2020

Codecov Report

Merging #588 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #588   +/-   ##
=========================================
  Coverage     55.31%   55.31%           
+ Complexity     9963     9962    -1     
=========================================
  Files          1032     1032           
  Lines         63371    63371           
  Branches       7159     7159           
=========================================
+ Hits          35051    35054    +3     
+ Misses        25842    25835    -7     
- Partials       2478     2482    +4     
Impacted Files Coverage Δ Complexity Δ
...ackend/influxdb/InfluxdbBackendListenerClient.java 39.55% <ø> (ø) 12.00 <0.00> (ø)
...a/org/apache/jmeter/timers/PoissonRandomTimer.java 72.97% <0.00%> (-5.41%) 9.00% <0.00%> (-1.00%)
...ache/jmeter/reporters/SummariserRunningSample.java 83.58% <0.00%> (-1.50%) 18.00% <0.00%> (-1.00%)
...n/java/org/apache/jmeter/reporters/Summariser.java 84.73% <0.00%> (-0.77%) 17.00% <0.00%> (-1.00%)
...main/java/org/apache/jorphan/gui/MenuScroller.java 34.73% <0.00%> (+4.19%) 10.00% <0.00%> (+2.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 cee4de9...9eb318d. Read the comment docs.

@asfgit asfgit closed this in f4447f0 May 5, 2020
@FSchumacher
Copy link
Contributor

@FSchumacher FSchumacher commented May 5, 2020

Thanks for the contribution. I hope you didn't mind my changes.

@bednar
Copy link
Contributor Author

@bednar bednar commented May 6, 2020

Thank you @FSchumacher

@bednar bednar deleted the docs-influxdb2 branch May 6, 2020
kkalinin pushed a commit to kkalinin/jmeter that referenced this issue Mar 11, 2021
Based on patch by Jakub Bednář.
Closes apache#588 on github
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants