Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Cache results in ReadDestination and ReadConsumerGroup metadata calls #337

Merged
merged 1 commit into from
Nov 30, 2017

Conversation

thuningxu
Copy link
Contributor

These are two most common metadata calls that actually hits Cassandra. Since destination and consumer group info does not change, let's cache it at the metadata_cassandra level.

@thuningxu thuningxu requested review from datoug and kirg November 30, 2017 03:04
@thuningxu thuningxu force-pushed the cache branch 3 times, most recently from 8c459e5 to e3b7529 Compare November 30, 2017 03:54
@coveralls
Copy link

coveralls commented Nov 30, 2017

Coverage Status

Coverage increased (+0.06%) to 66.859% when pulling 5d16549 on cache into d24994e on master.

@thuningxu thuningxu merged commit 371b784 into master Nov 30, 2017
@thuningxu thuningxu deleted the cache branch November 30, 2017 05:58
thuningxu pushed a commit that referenced this pull request Dec 6, 2017
thuningxu added a commit that referenced this pull request Dec 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants