Skip to content

GEODE-6529: Add puts operation timers#3347

Closed
kirklund wants to merge 7 commits intoapache:developfrom
kirklund:GEODE-6529-puts-timers
Closed

GEODE-6529: Add puts operation timers#3347
kirklund wants to merge 7 commits intoapache:developfrom
kirklund:GEODE-6529-puts-timers

Conversation

@kirklund
Copy link
Copy Markdown
Contributor

@kirklund kirklund commented Mar 21, 2019

Add the following puts operation timers:

  • replace
  • put-if-absent
  • put
  • create

Fix up CacheClientNotifierIntegrationTest to prevent hang.

Co-authored-by: Michael Oleske moleske@pivotal.io
Co-authored-by: Dale Emery demery@pivotal.io
Co-authored-by: Kirk Lund klund@apache.org

Please review: @demery-pivotal @mhansonp

kirklund and others added 6 commits March 21, 2019 15:26
Stub InternalCache.getMeterRegistry to prevent hang when meters are
added to LocalRegion

Co-authored-by: Michael Oleske <moleske@pivotal.io>
Co-authored-by: Kirk Lund <klund@apache.org>
* Add cache.region.operations.puts put timer.
* Measure calls to put operations.

Co-authored-by: Michael Oleske <moleske@pivotal.io>
Co-authored-by: Kirk Lund <klund@apache.org>
* Add cache.region.operations.puts put-if-absent timer.
* Measure calls to putIfAbsent operations.

Co-authored-by: Michael Oleske moleske@pivotal.io
Co-authored-by: Kirk Lund klund@apache.org
* Add cache.region.operations.puts replace timer.
* Measure calls to replace operations.

Co-authored-by: Michael Oleske moleske@pivotal.io
Co-authored-by: Kirk Lund klund@apache.org
Fix Mockito syntax and reorganize test.

Co-authored-by: Michael Oleske <moleske@pivotal.io>
Co-authored-by: Kirk Lund <klund@apache.org>
* Add cache.region.operations.puts create timer.
* Measure calls to create operations.

Co-authored-by: Michael Oleske <moleske@pivotal.io>
Co-authored-by: Dale Emery <demery@pivotal.io>
Co-authored-by: Kirk Lund <klund@apache.org>
@kirklund kirklund requested a review from moleske March 21, 2019 22:29
@kirklund kirklund closed this Mar 22, 2019
@kirklund kirklund deleted the GEODE-6529-puts-timers branch July 22, 2019 21:49
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.

2 participants