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

Feature release - initial version of metrics #574

Merged
merged 68 commits into from
May 27, 2024
Merged

Feature release - initial version of metrics #574

merged 68 commits into from
May 27, 2024

Conversation

novoj
Copy link
Collaborator

@novoj novoj commented May 27, 2024

No description provided.

novoj added 30 commits May 14, 2024 13:01
Initial implementation for transaction handling. Verified and extended the original concept of using JFR for metrics.
Initial implementation for transaction handling. Verified and extended the original concept of using JFR for metrics.
Initial implementation for transaction handling. Verified and extended the original concept of using JFR for metrics.
Observability is better wording for this part fo documentation.
Added basis of storage events. Made ReadOnlyHandle closeable and trying to consistently close it.
Observability is better wording for this part fo documentation.
Added monitoring of non-flushed data in OffsetIndexes.
Implemented transactional and storage metrics.
# Conflicts:
#	evita_external_api/evita_external_api_observability/src/main/java/io/evitadb/externalApi/observability/metric/provider/CustomEventProvider.java
#	evita_functional_tests/src/test/java/io/evitadb/utils/FileUtilsTest.java
#	evita_functional_tests/src/test/java/io/evitadb/utils/ReflectionLookupTest.java
Finalized query metrics.
Finalized session metrics.
Finalized session metrics.
Finalized background tasks and evitaDB start
Cache metrics done.
gRPC metrics done.
Corrected metrics name and initialization process.
Handle NULL labels safely.
Metrics finalization.
Also defined better default bucket count for Histograms.
novoj added 29 commits May 23, 2024 20:40
Also upgraded Undertow and XNIO to latest version to avoid security problems.
…hing-with-exception

fix(#572): NullPointerException in transactional stage
Unfortunately, Prometheus doesn't propagate native histograms in text format - see prometheus/prometheus#11265, only in Protobuff format, and this is not easily scrapeable.
It seems that the re-scheduling initiated an invocation storm (according to metrics). Tests added, logic tested and introduced minimal gap to avoid those storms.
@novoj novoj merged commit 5232f59 into master May 27, 2024
4 of 5 checks passed
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.

None yet

1 participant