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

[client-v2] Added documentation for OperationMetrics#completeOperation(). #1914

Open

chernser wants to merge 1 commit into main from v2_fix_metrics_npe

Contributor

chernser commented Nov 6, 2024

Summary

Added javadoc for com.clickhouse.client.api.metrics.OperationMetrics#operationComplete to warn about side effect if called multiple times.

See: #1900

Checklist

Delete items not relevant to your PR:

Unit and integration tests covering the common scenarios were added
A human-readable description of the changes was provided to include in CHANGELOG
For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials


          add documentation for OperationMetrics#completeOperation().

0e41e23

chernser mentioned this pull request

client-V2 com.clickhouse.client.api.metrics.Metric.getLong() #1900

Open

sonarcloud bot commented Nov 6, 2024

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet