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.

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

Enable node level metrics #621

Merged
merged 54 commits into from
Nov 15, 2023
Merged

Enable node level metrics #621

merged 54 commits into from
Nov 15, 2023

Conversation

amorton
Copy link
Contributor

@amorton amorton commented Nov 7, 2023

These metrics are more at the connection level with the node the driver thinks it is talking to.

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

kathirsvn and others added 30 commits October 23, 2023 14:52
@kathirsvn
Query Executor interface for CQL
ad94017
@maheshrajamani
[Native Protocol] Create and delete collection command changes (#581)
54ee2ec
@kathirsvn
Cache changes (#582)
02055dd
@maheshrajamani
[Native Protocol]Changes for InsertOperation (#586)
f74b0e8
@maheshrajamani
[Native Protocol] Read and count operation (#589)
40bc367
@kathirsvn
ReadAndUpdateOperation changes for native driver (#588)
3a858f5
@maheshrajamani
Fix for empty filter
136fa76
@maheshrajamani
Changes for delete command
67ccba4
@maheshrajamani
Changes for keyspace commands
f9836a9
@kathirsvn
CQL Session changes
44a19c7
@maheshrajamani
Fix for filter clauses
849e55c
@maheshrajamani
Merge branch 'native_cql_changes' of https://github.com/stargate/jsonapi
d5bc2a0 
 into native_cql_changes
@kathirsvn
Logging added
4c49fa8
@maheshrajamani
Fixed code for boolean sort
83694ad
@maheshrajamani
Merge branch 'native_cql_changes' of https://github.com/stargate/jsonapi
82a6c2d 
 into native_cql_changes
@kathirsvn
CQL session fix
bea0649
@kathirsvn
Merge branch 'native_cql_changes' of github.com:stargate/jsonapi into…
2f46109 
… native_cql_changes
@kathirsvn
CQL session fix
0832de8
@maheshrajamani
Set default value to null so no need to serialize the key
a1abb8a
@maheshrajamani
Merge branch 'native_cql_changes' of https://github.com/stargate/jsonapi
d39c199 
 into native_cql_changes
@kathirsvn
CQL session cache key fix
94d98f6
@kathirsvn
Merge branch 'native_cql_changes' of github.com:stargate/jsonapi into…
0575802 
… native_cql_changes
@kathirsvn
CQL session cache key fix
92e0e85
@maheshrajamani
IT bug fixes
b874e52
@maheshrajamani
Avoid creating bridge Values
983420c
@kathirsvn
Moved cache configurations to quarkus properties
70944a2
@kathirsvn
CqlSessionCache key changes
bdbe639
@kathirsvn
Merge branch 'native_cql_changes' of github.com:stargate/jsonapi into…
d806343 
… native_cql_changes
@kathirsvn
CQL session cache eviction listener added
e6090b5
@kathirsvn
getSessionCacheKey method improvement
873193c
kathirsvn and others added 21 commits October 30, 2023 12:33
@kathirsvn
Changes to use token as cache key for OSS cassandra when present in t…
d640c0a 
…he request
@kathirsvn
Metrics changes
d1ab1cd
@kathirsvn
Fixed Auth token changes for IT
6f44428
@kathirsvn
Fixed driver metrics conf
90ffc3f
@maheshrajamani
Added histogram metrics
9dda913
@maheshrajamani
Removed run on subscription
873e04f
@maheshrajamani
Added http request histogram
57d465a
@kathirsvn
JAVA_OPTIONS change
13d83a7
@Yuqi-Du
cql branch, Error mapping, IT fix (#610)
77f47af
@kathirsvn
JAVA_OPTIONS change reverted
89d8024
@kathirsvn
Merge branch 'native_cql_changes' of github.com:stargate/jsonapi into…
b87a21f 
… native_cql_changes
@kathirsvn
Session metrics id generator changes
470bf43
@maheshrajamani
Fixed the unit test for metrics to ignore histogram metrics
898042b
@maheshrajamani
Merge branch 'native_cql_changes' of https://github.com/stargate/jsonapi
ad7260e 
 into native_cql_changes
@tatu-at-datastax
Merge changes from main to native_cql_changes (#615)
54f9c34
@tatu-at-datastax
Merge branch 'main' into native_cql_changes (resolve conflicts)
44ad4c5
@Yuqi-Du
restore DeleteOneCommandResolverTest.java
6f073d4
@Yuqi-Du @tatu-at-datastax
merge main to native_cql_branch (#616)
7c66e93 
Co-authored-by: Tatu Saloranta <tatu.saloranta@datastax.com>
@tatu-at-datastax
Fix #617: increase Java driver local connection pool size to 8
69f16c3
@tatu-at-datastax
Merge branch 'main' into native_cql_changes
43f4af3
@amorton
Expose driver node metrics
7cb8321 
These metrics are more at the connection level with the node
the driver things it is talking to.
@amorton amorton requested a review from a team as a code owner November 7, 2023 21:31
Base automatically changed from native_cql_changes to main November 15, 2023 18:21
@amorton
Copy link
Contributor Author

amorton commented Nov 15, 2023

checked after jeff merged main on, looks good.

@jeffreyscarpenter jeffreyscarpenter merged commit de5180d into main Nov 15, 2023
3 checks passed
@jeffreyscarpenter jeffreyscarpenter deleted the ajm-node-metrics branch November 15, 2023 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yuqi-Du Yuqi-Du Yuqi-Du approved these changes

@kathirsvn kathirsvn kathirsvn approved these changes

@jeffreyscarpenter jeffreyscarpenter jeffreyscarpenter approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@amorton @jeffreyscarpenter @kathirsvn @Yuqi-Du @maheshrajamani @tatu-at-datastax