Skip to content
This repository was archived by the owner on Nov 17, 2020. It is now read-only.

cache topic access #217

Merged
merged 2 commits into from
Nov 7, 2019
Merged

Conversation

velimir
Copy link
Contributor

@velimir velimir commented Nov 7, 2019

Proposed Changes

Backport AMQP channel topic access caching (performance optimisation).

Types of Changes

  • Bugfix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, appearance)

Checklist

  • I have read the CONTRIBUTING.md document
  • I have signed the CA (see https://cla.pivotal.io/sign/rabbitmq)
  • All tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in related repositories

Further Comments

Original

channel - original

connection - original

1 users (always hit the cache)

channel - topic access

connection - topic access

@michaelklishin michaelklishin merged commit 1c9e9d7 into rabbitmq:master Nov 7, 2019
michaelklishin added a commit that referenced this pull request Nov 7, 2019
cache topic access

(cherry picked from commit 1c9e9d7)
michaelklishin added a commit that referenced this pull request Nov 7, 2019
cache topic access

(cherry picked from commit 1c9e9d7)
@michaelklishin michaelklishin added this to the 3.7.22 milestone Nov 8, 2019
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.

2 participants