spacewander commented on a change in pull request #3987: URL: https://github.com/apache/apisix/pull/3987#discussion_r607637962
########## File path: docs/en/latest/router-radixtree.md ########## @@ -116,9 +118,9 @@ curl http://127.0.0.1:1980/hello All requests only hit the route of port `1980`. -2. Different routes have the same uri and set different matching conditions +2. Different routes have the same `uri` and set `different matching conditions` Review comment: Why quote "different matching conditions"? ########## File path: docs/en/latest/router-radixtree.md ########## @@ -71,11 +71,13 @@ Here are the rules: |/blog/foo/gloo | `/blog/foo/*` | |/blog/bar | not match | -#### 4. Different routes have the same uri +#### 4. Different routes have the same `uri` -When different routes have the same uri, you can set the priority attribute of the route to determine which route to match first, or add matching rules to distinguish different routes. +When different routes have the same uri, you can set the priority field of the route to determine which route to match first, or add other matching rules to distinguish different routes. -1. Different routes have the same uri and set the `priority` attribute +Note: In matching rules, the `priority` field takes precedence over other rules. Therefore, we can set other matching rules only when the value of the `priority` field is the same. Review comment: > Therefore, we can set other matching rules only when the value of the `priority` field is the same. Can't understand it. We can set `hosts` regardless of the value of `priority`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
