Skip to content

[type:refactor] remove shenyu trie cache - #6303

Merged
Aias00 merged 12 commits into
apache:masterfrom
moremind:remove-trie
Mar 27, 2026
Merged

Aias00 merged 12 commits into
apache:masterfrom
moremind:remove-trie

Conversation

@moremind

@moremind moremind commented Mar 13, 2026 •

Copy link
Copy Markdown
Member

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@moremind moremind added this to the 2.7.1 milestone Mar 15, 2026
@Aias00
Aias00 merged commit 295a12b into apache:master Mar 27, 2026
41 checks passed
Aias00 pushed a commit that referenced this pull request Oct 26, 2026
…moval (#8117)

The Trie L2 cache was removed in #6303, but comments still described the
L1/L2/Trie lookup chain. Describe the actual matching path instead: the
bounded per-path match cache followed by default matching on a miss.

Also drop the remaining Trie references from maintained matching docs:
remove the orphaned L2 config comment, rename the L1 cache comments to
match cache, fix the trie wording in CommonPluginDataSubscriber and
LocalPluginController, and point HttpRouteParser at the current
matchSelector method.

Historical references in RELEASE-NOTES.md are kept.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants