Firstsawyou commented on a change in pull request #3987:
URL: https://github.com/apache/apisix/pull/3987#discussion_r607708875
##########
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:
Yes, this sentence seems superfluous.
Updated.
##########
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:
updated.
--
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]
